Skip to content

Enhance workflow to label external PRs - #22381

Merged
nil-is-all merged 1 commit into
mainfrom
nil-is-all-patch-8
Sep 2, 2026
Merged

nil-is-all merged 1 commit into
mainfrom
nil-is-all-patch-8

Conversation

@nil-is-all

Copy link
Copy Markdown
Contributor

Updated the workflow to label external PRs with a community label and refined the exclusion list for authors and bots.

We already have a label - community: contribution. Just updating the workflow which adds community contributed PRs to the Project Board to have this label.

Updated the workflow to label external PRs with a community label and refined the exclusion list for authors and bots.
@pytorch-bot

pytorch-bot Bot commented Sep 1, 2026 •

Copy link
Copy Markdown

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/22381

Note: Links to docs will display an error until the docs builds have been completed.

✅ You can merge normally! (3 Unrelated Failures)

As of commit 4de76db with merge base f5102b8 (image):

BROKEN TRUNK - The following jobs failed but were present on the merge base:

👉 Rebase onto the `viable/strict` branch to avoid these failures

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@meta-cla meta-cla Bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Sep 1, 2026
@nil-is-all nil-is-all added the release notes: none Do not include this in the release notes label Sep 1, 2026
"zhenyan-zhang-meta", "silverguo", "harishs88ss", "AlannaBurke", "Doggeral", "laithsakka", "Reubend", "dbort", "huydhn", "mcremon-meta",
"trivedivivek", "angelayi", "helunwencser", "hsharma35", "zhxchen17", "iseeyuan", "svekars", "nathanaelsee", "dulinriley",
"jerryzh168", "cmodi-meta", "bigfootjon", "sxu", "ydwu4", "Riandy", "tugsbayasgalan", "bsoyluoglu", "yangw-dev",
"nil-is-all", "tanvirislam-meta", "cbilgin", "kimishpatel", "psiddh", "digantdesai", "SS-JIA", "ahmtox", "mcr229",

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is there a way to check the org for the author or something easier to maintain?

]);

// List of organization logins (lowercased) to exclude members of
const excludedOrgs = new Set([

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@digantdesai yes, I have also added a check for Orgs. The initial version of this automation manually had the author names to exclude (as above). I continued to keep track of individual authors, but can also clean it up to just exclude based on Org if that is easier to maintain

@nil-is-all
nil-is-all merged commit 457a2a8 into main Sep 2, 2026
68 of 70 checks passed
@nil-is-all
nil-is-all deleted the nil-is-all-patch-8 branch September 2, 2026 17:31
nil-is-all added a commit that referenced this pull request Sep 28, 2026
Instead of using a Personal Access Token, a GitHub app with read and
write permissions for this repo was created. Added the client-id as a
variable: EXECUTORCH_TRIAGE_APP_CLIENT_ID and private-key was stored in
EXECUTORCH_TRIAGE_APP

Part of adding "community: contribution" label to OSS issues and PRs
(#22381)

This branch was previously deployed

1 inactive deployment
cadence — 4de76db0 Deployed Sep 1, 2026 by nil-is-all
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. release notes: none Do not include this in the release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants