Skip to content

Conversation

maelle
Copy link
Contributor

@maelle maelle commented Jul 22, 2025

Fix #2004

🔥

@maelle
Copy link
Contributor Author

maelle commented Jul 22, 2025

Let's also wait for a third opinion.

@maelle
Copy link
Contributor Author

maelle commented Aug 19, 2025

@krlmlr are you ok with this change?

@krlmlr
Copy link
Contributor

krlmlr commented Aug 21, 2025

Is there an easy way to get a static preview of the demos, perhaps as a Quarto document? We could then decide if we want to reuse this content.

To avoid blockers, we could remove the igraphdemo() function first, in a separate PR.

@schochastics
Copy link
Contributor

imho we can delete all demos directly. I looked at the files and the content is already covered in the vignettes I am currently writing.

@maelle
Copy link
Contributor Author

maelle commented Aug 27, 2025

So is it fine to merge @krlmlr?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Retire the demos?
3 participants