-
Notifications
You must be signed in to change notification settings - Fork 13.1k
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
feat: auto-label PRs that contain db migrations #26745
Conversation
Nice. Looks like it worked. Removing the DUMMY file now, and this should be good to merge. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #26745 +/- ##
==========================================
- Coverage 69.56% 67.27% -2.30%
==========================================
Files 1892 1892
Lines 74162 74162
Branches 8263 8263
==========================================
- Hits 51593 49894 -1699
- Misses 20488 22187 +1699
Partials 2081 2081
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Thanks for the PR @mistercrunch. This will be really valuable when cherry-picking fixes into patch releases as we avoid PRs that contain migrations. There's a typo here label-miration-pr.yml. Could you change it to |
Ok, fixed the typo, should be good to merge |
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
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.
approving as codeowner
The goal here is simply to auto-assign a GitHub label
risk:db-migration
to every PR that touches that folder. Having a label we can rely on is good so it can be trusted for release automation and other purposes.I'm closing/reopening #26683 while pointing to the
apache
repo as opposed to a fork since security / perms work only on the main fork apparently.