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 use of test_broken on Julia 1.10 #297

Merged
merged 2 commits into from
Aug 8, 2024
Merged

Fix use of test_broken on Julia 1.10 #297

merged 2 commits into from
Aug 8, 2024

Conversation

iamed2
Copy link
Collaborator

@iamed2 iamed2 commented Aug 8, 2024

Julia 1.10 was failing tests before this.

There may have been issues with the value of parsed being reused in the loop, so they're in a let block now, and an error in the func call will skip the tests that depend on it. It's a bit hacky but I believe this is all using standard behaviour for tests, and it tests the behaviour it needs to test.

@iamed2 iamed2 merged commit 4752ef0 into master Aug 8, 2024
14 of 15 checks passed
@iamed2 iamed2 deleted the ed/fix-test_broken branch August 8, 2024 19:22
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.

1 participant