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

ci: add workaround for failing CI on Windows #191

Merged
merged 1 commit into from
Jul 3, 2024

Conversation

JKRhb
Copy link
Contributor

@JKRhb JKRhb commented May 27, 2024

As mentioned in #189, there is currently an issue with the CI when it is running under Dart 3.4.0 and Windows. This issue adds a workaround that lets the CI fall back to 3.3.0 under Windows so that the respective job does not fail anymore due to a timeout. As soon as the issue has been fixed upstream, this change can be reverted.

@JKRhb JKRhb force-pushed the windows-ci-workaround branch from ee961ef to 60761bb Compare May 27, 2024 19:11
@JKRhb
Copy link
Contributor Author

JKRhb commented May 27, 2024

Hmmm. For some reason, lowering the version to 3.3.0 on Windows does not help here 🤔 It's a bit strange since in another project, that actually made the CI pass again. I will investigate why that might be the case.

@shamblett
Copy link
Owner

Yep thanks, I've seen weird behaviour like this before over the years, usually more in browser tests, sometimes it just seems to go away, also in one instance upgrading to the latest version of the test package cleared the hang.

@JKRhb JKRhb force-pushed the windows-ci-workaround branch from 60761bb to 44fffb6 Compare July 2, 2024 18:50
@JKRhb
Copy link
Contributor Author

JKRhb commented Jul 2, 2024

Sorry, I realized only now that there was an essential line missing for this workaround to actually work 🤦 Now this PR should be ready for merging :)

@shamblett shamblett merged commit d6b67d6 into shamblett:master Jul 3, 2024
3 checks passed
@JKRhb JKRhb deleted the windows-ci-workaround branch July 3, 2024 08:11
@shamblett
Copy link
Owner

Thanks, package re released at version 9.0.5

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