Skip to content
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

Fix breakage after base unexported at-with #62

Merged
merged 1 commit into from
Mar 31, 2024
Merged

Conversation

yuyichao
Copy link
Contributor

@palday
Copy link

palday commented Mar 23, 2024

Would be great to get this merged and released because it's leading to MixedModels.jl tests failing on 1.11 and nightly.

Copy link

@joachimbrand joachimbrand left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like the right fix.
It would be good to have this merged soon as Rimu.jl is failing CI on nightly due to this issue.

@MilesCranmer
Copy link

Thanks; this was breaking some CI.

Also, why was that merged into JuliaLang? Doesn't seem to follow SemVer?

@palday
Copy link

palday commented Mar 24, 2024

@MilesCranmer 1.11 isn't yet released and this only became available on 1.11, so now is the time to make such changes.

@MilesCranmer
Copy link

Ping, can this be merged?

@yuyichao
Copy link
Contributor Author

Seems that #63 should fix the error for the latest version although as clarified in JuliaLang/julia#53628 the correct fix should be to use Base.ScopedValues explicitly (as in this PR) rather than to use Base.@with explicitly.

@yuyichao yuyichao merged commit 8aeedc6 into master Mar 31, 2024
14 checks passed
@yuyichao yuyichao deleted the yyc/scopedvalues branch March 31, 2024 14:42
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