-
Notifications
You must be signed in to change notification settings - Fork 120
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
Start fixing flakey tests ❄️ #3329
Conversation
Generated by 🚫 Danger Swift against 134b8f4 |
❌ 3 Tests Failed:
View the top 2 failed tests by shortest run time
View the full list of 1 ❄️ flaky tests
To view individual test run time comparison to the main branch, go to the Test Analytics Dashboard |
447b7b6
to
e51a103
Compare
b4c5d2f
to
98d1cc7
Compare
|
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.
Very very nice! Fingers crossed 🤞
First round of changes and a non-failing run. Probably best to merge this so the Preview Tests are more stable and look at other flakey unit tests separately.
The PR has individual commits for each change.