Skip to content

update CI jobs #101

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 8 commits into from
Mar 15, 2025
Merged

update CI jobs #101

merged 8 commits into from
Mar 15, 2025

Conversation

aplavin
Copy link
Member

@aplavin aplavin commented Mar 15, 2025

The goal is to make all CI tests pass, so that logs from other PRs can be assessed more easily (specifically #99 and #100).

@aplavin
Copy link
Member Author

aplavin commented Mar 15, 2025

Setfield tests seem to be failing on 1.10 and 1.11. I guess it's just a matter of too aggressive timing comparison... Is it ok to remove Setfield on 1.10+ from integration tests for now, @jw3126?

Flatten appears to be genuinely broken on 1.11. Is it ok to remove Flatten on 1.11+ from integration tests for now, @rafaqz?

@rafaqz
Copy link
Member

rafaqz commented Mar 15, 2025

Yeah noticed these, a lot of changes in Base

Flatten.jl is still apparently working in practice where it's used, so may also be some tests like thosr in Setfield. It just hasn't had any new PRs on 1.11.

I'll try to look at it this week it's good to have tested here

@aplavin
Copy link
Member Author

aplavin commented Mar 15, 2025

I guess I'll merge as-is, we'll just keep in mind these three failures for now. It is definitely an improvement already.

@aplavin aplavin merged commit b987640 into master Mar 15, 2025
21 of 24 checks passed
@aplavin aplavin deleted the ci-versions branch March 15, 2025 14:49
@rafaqz
Copy link
Member

rafaqz commented Mar 15, 2025

Thanks

@jw3126
Copy link
Member

jw3126 commented Mar 15, 2025

Thanks. Setfield.jl can be removed in future.

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.

3 participants