Skip to content

Remove Julia v1.8 version check for precompilation #189

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 1 commit into from
Jul 27, 2025

Conversation

ChrisRackauckas
Copy link
Member

Summary

  • Removed the version check for Julia v1.8 in the precompilation code
  • The precompilation file is now included unconditionally

Motivation

Since the current Julia LTS is v1.10, version checks for v1.8 are no longer necessary. This simplifies the code by removing conditional compilation that all supported Julia versions will execute anyway.

Changes

  • Removed @static if VERSION >= v"1.8" check around include("precompile.jl")
  • Updated comment to reflect that precompilation is now always enabled

Test plan

  • All existing tests pass without modification
  • Package precompiles successfully on Julia 1.6+ (the minimum supported version per Project.toml)

🤖 Generated with Claude Code

@ChrisRackauckas ChrisRackauckas force-pushed the remove-julia-1.8-version-check branch from a01c3e4 to 4f13604 Compare July 27, 2025 16:07
Since the current Julia LTS is v1.10, we can enable precompilation for
all supported Julia versions (1.6+). This removes the version check that
was preventing Julia 1.6 and 1.7 users from benefiting from precompilation.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@ChrisRackauckas ChrisRackauckas force-pushed the remove-julia-1.8-version-check branch from 4f13604 to f58adf4 Compare July 27, 2025 16:24
@ChrisRackauckas ChrisRackauckas merged commit c84cbc9 into main Jul 27, 2025
8 of 9 checks passed
@ChrisRackauckas ChrisRackauckas deleted the remove-julia-1.8-version-check branch July 27, 2025 23:48
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