-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Seperate mutation tests #2331
Seperate mutation tests #2331
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2331 +/- ##
=======================================
Coverage 85.40% 85.40%
=======================================
Files 313 313
Lines 7467 7467
Branches 1126 1126
=======================================
Hits 6377 6377
Misses 745 745
Partials 345 345
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
ed70307
to
07aa44d
Compare
07aa44d
to
260bc68
Compare
Co-authored-by: Martin Costello <[email protected]>
Just one final edit please, then this should be good to go. I'll just need to get the required statuses updated. |
Co-authored-by: Martin Costello <[email protected]>
@joelhulen Could you edit the branch protections to make the following required please?
|
Thanks again for this @ShawnWu33 - this shaves a good 10-15 minutes off the total time between opening a PR and being able to merge it now. |
Pull Request
The issue or feature being addressed
Resolves #1755
Details on the issue fix or feature implementation
Tested these three workflow in my repo, below is what they lookslike during execution:
Build workflow: https://github.com/ShawnWu33/Polly/actions/runs/11216605547
Mutation core workflow: https://github.com/ShawnWu33/Polly/actions/runs/11216605554
Mutation legacy workflow: https://github.com/ShawnWu33/Polly/actions/runs/11216605545
Confirm the following