Skip to content

Issue #399#425

Open
franzi448 wants to merge 6 commits into
JuliaDynamics:mainfrom
franzi448:Issue-#399
Open

Issue #399#425
franzi448 wants to merge 6 commits into
JuliaDynamics:mainfrom
franzi448:Issue-#399

Conversation

@franzi448
Copy link
Copy Markdown
Contributor

Remove 'Requires' and bump to 1.9

@Datseris
Copy link
Copy Markdown
Member

@franzi448 this is not correct. You will need to consult this page to properly make package extensions work in Julia 1.9: https://pkgdocs.julialang.org/v1/creating-packages/#Conditional-loading-of-code-in-packages-(Extensions)

there is no @require anywhere in the pure Julia version.

@franzi448
Copy link
Copy Markdown
Contributor Author

Sorry, thank you for the link! I tried again and hope this time it is right.

@Datseris
Copy link
Copy Markdown
Member

Datseris commented Oct 4, 2024

I am sorry for being late here. Thanks for your continued effort! Perhaps you can work on making sure all tests pass? Both the formal tests and the documentation build. It is much better for me to review a PR whose tests are passing.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants