-
Notifications
You must be signed in to change notification settings - Fork 156
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
Test sharding #4912
base: master
Are you sure you want to change the base?
Test sharding #4912
Conversation
Does the PR have any schema changes?Looking good! No breaking changes found. Maintainer note: consult the runbook for dealing with any breaking changes. |
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.
This is awesome!! 🤩
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #4912 +/- ##
=======================================
Coverage 24.41% 24.41%
=======================================
Files 360 360
Lines 143270 143270
=======================================
Hits 34984 34984
Misses 108187 108187
Partials 99 99 ☔ View full report in Codecov by Sentry. |
This file was missing a warning that it is auto-generated. Fixed now in https://github.com/pulumi/ci-mgmt/pull/1231/files @blampe what's next for this PR can we close? I assume once the changes make it to ci-mgmt and your latest fixes to AWS specifics this can "naturally" be updated through ci-mgmt? |
@t0yv0 yes, after pulumi/ci-mgmt#1217 merges we will be able to set I marked this as a draft since I'll be re-using it occasionally to confirm I haven't broken anything. |
No description provided.