Skip to content

use [sources] #2699

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
May 16, 2025
Merged

use [sources] #2699

merged 3 commits into from
May 16, 2025

Conversation

oscardssmith
Copy link
Member

This is the OrdinaryDiffEq followup to SciML/NonlinearSolve.jl#571.

@ranocha
Copy link
Member

ranocha commented May 5, 2025

Looking at https://pkgdocs.julialang.org/v1.10/toml-files/ and https://pkgdocs.julialang.org/v1.11/toml-files/, it seems like the sources section is only available in Julia 1.11, not in the LTS 1.10. Is this correct?

@oscardssmith
Copy link
Member Author

that is correct. on 1.10 and under, this change won't do anything (but everything will still work)

@ranocha
Copy link
Member

ranocha commented May 5, 2025

Thanks! Just wanted to confirm this.

@oscardssmith
Copy link
Member Author

@avik-pal what did you do to make this work for CI on NonlinearSolve?

@avik-pal
Copy link
Member

it pretty much just worked 😅. NonlinearSolve does have a per subpackage CI workflow. Can you try and change https://github.com/oscardssmith/OrdinaryDiffEq.jl/blob/7a35128936dd2fbbf759db93545cf87b3fc0a867/.github/workflows/CI.yml#L84 to run only on 1.10?

@oscardssmith
Copy link
Member Author

lets try it...

@oscardssmith oscardssmith merged commit a4c3b60 into SciML:master May 16, 2025
1 check passed
@oscardssmith
Copy link
Member Author

that seems to work. Thanks!

@oscardssmith oscardssmith deleted the os/use-sources branch May 16, 2025 14:34
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.

4 participants