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 @atomic usage on 1.7-1.9 #127

Merged

Conversation

topolarity
Copy link
Contributor

Fixes a regression from #125

This also adds 1.9 to the CI since apparently that was missing.

@topolarity topolarity changed the title Fix @atomic usage on 1.9 Fix @atomic usage on 1.7-1.9 Jan 22, 2024
@DhairyaLGandhi
Copy link
Collaborator

Looks great! Thanks @topolarity ! Note to self: We can retain the ci for now, but release a breaking release which strips out the code for older Julia versions.

@DhairyaLGandhi DhairyaLGandhi merged commit c8df0b7 into JuliaIO:master Jan 22, 2024
10 checks passed
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