- Notifications
You must be signed in to change notification settings - Fork329
ci: update wasmbrowsertest to a specific commit#514
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
Using latest wasn't getting the latest version of wasmbrowsertest, sowe're now using a specific commit.Current version was failing due to timeouts, so we're updating tolatest commit to see if it fixes the issue.
This repo had a similar issue, and it looks like they fixed it by updating the version as well:hajimehoshi/ebiten#2982 |
This problem looks like a flaky test, so I'm not entirely sure if this PR will actually fix the issue. The CI for this PR (#509) has now passed, even though |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Weird, issue. Thanks for looking into it, I think it's good to lock down the version anyway, so let's hope this fixes it 🤞🏻
d1468a7 intocoder:masterUh oh!
There was an error while loading.Please reload this page.
Using latest wasn't getting the latest version of wasmbrowsertest, so we're now using a specific commit.
Current version was failing due to timeouts, so we're updating to latest commit to see if it fixes the issue.