-
Notifications
You must be signed in to change notification settings - Fork 30
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
chore: add workflow to check if change file is included in a PR #887
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## dev #887 +/- ##
===========================================
+ Coverage 38.75% 62.81% +24.06%
===========================================
Files 282 282
Lines 10922 10922
Branches 1517 1517
===========================================
+ Hits 4233 6861 +2628
+ Misses 6382 3523 -2859
- Partials 307 538 +231 ☔ View full report in Codecov by Sentry. |
a70a0d2
to
5be68d8
Compare
how will this work with release prs? #884 these dont have "release not needed" on them, yet change files are not required |
bcebbcf
to
458212b
Compare
Good catch @gcbeattyAWS! I updated the condition to skip PRs with the label 'Release PR' |
Description of changes:
The PR updates the CONTRIBUTION.md file and adds a new Workflow that checks if a PR contains a
change
file. If a change file is not required, PRs should add the labelRelease Not Needed
.By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.