-
Notifications
You must be signed in to change notification settings - Fork 1
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
Bumps webdriverio
to latest, adds isRetryableError
utility, reduces test flakiness
#89
Conversation
@romainmenke I didn't see any |
Thank you for spending time on this @mhassan1 🙇 Should we also retry on this error: https://github.com/mrhenry/polyfill-library/actions/runs/12148384546/job/33876735290
Or possibly extending the sharding of tests to IE 10 and IE 11? It is mostly important that we test each feature individually with every other polyfill present. Not so much that we test all features in one go. |
webdriverio
to latest, adds retry logic on failure to create browserstack sessionwebdriverio
to latest, adds isRetryableError
utility
@romainmenke I've created an |
Nice! https://github.com/mrhenry/polyfill-library/actions/runs/12153971340/job/33892964196 Maybe this one too?
|
Added. |
- better naming for test runs - shard ie/10
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.
LGTM!
Thank you for working on this 🙇
webdriverio
to latest, adds isRetryableError
utilitywebdriverio
to latest, adds isRetryableError
utility, reduces test flakiness
This PR bumps
webdriverio
to latest, which may help with CI flakiness. It also adds anisRetryableError
utility that will retry jobs on the following errors:It also includes #90, which further reduces test flakiness.