Add Julia 1.11 testing and exclude Enzyme tests from Julia 1.12 prerelease #2905
+16
−0
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…lease This commit makes two improvements to the CI configuration: 1. Adds explicit Julia 1.11 version to the test matrix to ensure 1.11 continues to be tested even as newer versions become the latest stable. 2. Excludes the Enzyme test group from running on Julia 1.12 prerelease versions using matrix exclusion. This prevents unnecessary CI jobs from being created, as Enzyme tests are already conditionally skipped on prerelease versions in test/runtests.jl:179. These changes optimize CI resources while ensuring proper test coverage across Julia versions. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Added defensive guards at the top of Enzyme test files to explicitly skip tests on Julia 1.12+ prerelease versions. This prevents accidental Enzyme loading if these test files are run directly outside of the normal test harness. While the CI workflow and test/runtests.jl already prevent Enzyme tests from running on v1.12, these file-level guards provide an additional safety layer and make the version requirement explicit in each test file. Affected files: - test/enzyme/autodiff_events.jl - test/enzyme/discrete_adjoints.jl 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Author
Additional ChangesAdded defensive version guards to Enzyme test files to prevent accidental Enzyme loading on Julia 1.12+:
These guards provide an additional safety layer beyond the existing controls in:
Summary of all Enzyme-related version guards in the codebase:Already properly guarded:
Newly guarded:
All test files that use or could load Enzyme now have proper version guards to prevent loading on Julia 1.12+ prerelease versions. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Changes
This PR modifies
.github/workflows/CI.ymlto:'1.11'to the version matrix (line 68)'pre'(Julia 1.12 prerelease) versions (lines 71-73)Rationale
The Enzyme tests are already conditionally skipped on prerelease versions in
test/runtests.jl:179, so excluding them at the matrix level prevents unnecessary CI jobs from being created and saves CI resources.Test plan
🤖 Generated with Claude Code