CI: Automatically add 'Needs Review' label to newly opened PRs#19959
CI: Automatically add 'Needs Review' label to newly opened PRs#19959aladdin-afk wants to merge 1 commit intoankidroid:mainfrom
Conversation
david-allison
left a comment
There was a problem hiding this comment.
pull_request_target) and bugs here can cause catastrophic damage to the repo.
Please rebase this so there is only 1 commit, and that the commit only contains necessary changes.
Please list the docs |
56fe8ed to
6e13d37
Compare
This comment was marked as outdated.
This comment was marked as outdated.
BrayanDSO
left a comment
There was a problem hiding this comment.
How Has This Been Tested?
Verified the workflow logic in YAML file.
Confirmed the job only triggers for newly opened, undrafted prs
Please try the workflow on your fork, then report back the results
|
This needs squashing before it's reviewed |
d4df074 to
f0b7671
Compare
Roger That |
the logic i wrote s correct, but unable to reproduce in my fork. |
|
sry uni, opensource been a tough balance, respecting my parents i have take the hard decision. |
Purpose / Description
Automated adding the Needs Review label to new opened, nondraft prs. This removes the need for maintainers to be pinged or check manually.
Fixes
Approach
Added a new job that uses github script to automatically add the Needs Review label when a new pr is opened or is not a draft.
How Has This Been Tested?
Verified the workflow logic in YAML file.
Confirmed the job only triggers for newly opened, undrafted prs
Learning (optional, can help others)
referenced GitHub Actions documentation for workflow conditions.
https://docs.github.com/en/actions/reference/workflows-and-actions/workflow-syntax
https://docs.github.com/en/actions/reference/workflows-and-actions/events-that-trigger-workflows
Checklist
Please, go through these checks before submitting the PR.