Skip to content

Allow functioning on Julia 1.10#7

Closed
oscardssmith wants to merge 3 commits into
mainfrom
os/lower-julia-compat
Closed

Allow functioning on Julia 1.10#7
oscardssmith wants to merge 3 commits into
mainfrom
os/lower-julia-compat

Conversation

@oscardssmith
Copy link
Copy Markdown
Member

@Shreyas-Ekanathan any specific reason why this was requiring 1.11 previously?

@Shreyas-Ekanathan any specific reason why this was requiring 1.11 previously?
@Shreyas-Ekanathan
Copy link
Copy Markdown
Contributor

not that I know of, I think it just built like that

@oscardssmith
Copy link
Copy Markdown
Member Author

Also, @Shreyas-Ekanathan can you fix the docs build?

@Shreyas-Ekanathan
Copy link
Copy Markdown
Contributor

I don't think I have perms to contribute to this pr, but if you go to docs/project.toml and replace the uuid of SparseBandedMatrices with "bd59d7e1-4699-4102-944e-d05209cb92aa" it should work I think.

@ChrisRackauckas-Claude
Copy link
Copy Markdown

PR Analysis

I've analyzed this PR and tested it locally. Here's the summary:

Changes in this PR

  • Fixes the UUID in docs/Project.toml from an incorrect UUID to the correct one (bd59d7e1-4699-4102-944e-d05209cb92aa)
  • This enables Julia 1.10 support

CI Status

The CI checks shown are from September 2025 (the logs have expired). The failures visible are:

  1. Documentation build - Failed due to incorrect UUID (now fixed in this PR)
  2. Format-check - Pre-existing issue that also fails on main (JuliaFormatter/SciMLStyle installation issue)
  3. Spell Check - Pre-existing issue

Local Testing

I ran the tests locally after merging main into this branch and all 27 tests pass on Julia 1.11.

Action Needed

The branch needs to be updated with current main to trigger fresh CI. Since I don't have push access to this branch, could someone with access:

  1. Merge main into this branch: git merge origin/main
  2. Push to trigger a new CI run

Alternatively, a maintainer can merge this PR directly since:

  • The UUID fix is correct
  • Tests pass locally
  • The format-check and spell check failures are pre-existing issues on main

cc @ChrisRackauckas

🤖 Generated with Claude Code

@ChrisRackauckas ChrisRackauckas deleted the os/lower-julia-compat branch January 11, 2026 21:05
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