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

Restore cmdlineargs tests on non-Windows platforms #55368

Merged
merged 1 commit into from
Aug 7, 2024

Conversation

giordano
Copy link
Contributor

@giordano giordano commented Aug 4, 2024

These tests have been flaky for some time (#46537, #49154) and were skipped in #50071, but should be working better now according to #46537 (comment).

@giordano giordano added multithreading Base.Threads and related functionality ci Continuous integration labels Aug 4, 2024
@giordano giordano marked this pull request as draft August 4, 2024 14:05
@giordano giordano marked this pull request as ready for review August 4, 2024 14:05
These tests have been flaky for some time, but should be working better now.
@giordano giordano force-pushed the mg/test-cmdlineargs branch from cc6b988 to 3c4425b Compare August 4, 2024 14:07
@IanButterworth IanButterworth requested a review from vtjnash August 4, 2024 23:53
@giordano
Copy link
Contributor Author

giordano commented Aug 6, 2024

I stress tested this PR by running the test ten times over a few days, and the errors reported in #46537 and #49154 didn't show up at any point (there were other intermittent failures, but not the ones in cmdlineargs). It sounds like the underlying issue was indeed fixed, as suggested by #46537 (comment).

@IanButterworth IanButterworth merged commit b3a62b4 into JuliaLang:master Aug 7, 2024
7 checks passed
@giordano giordano deleted the mg/test-cmdlineargs branch August 7, 2024 05:16
lazarusA pushed a commit to lazarusA/julia that referenced this pull request Aug 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci Continuous integration multithreading Base.Threads and related functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants