Skip to content

Label allowlist pull requests - #311

Merged
v-robaiken merged 4 commits into
mainfrom
robaiken/label-allowlist-prs
Oct 12, 2026
Merged

v-robaiken merged 4 commits into
mainfrom
robaiken/label-allowlist-prs

Conversation

@v-robaiken

@v-robaiken v-robaiken commented Oct 12, 2026 •

Copy link
Copy Markdown
Contributor

Adds an issue labeler workflow modeled on dependabot-core's. It applies the allowlist label to pull requests whose title or description mentions "allowlist" or "allow-list".

Note: github/issue-labeler matches regexes against the title and description text, not changed files. Tested against the last 80 PRs: it labels all 23 PRs that changed internal/handlers/egress_allowlist_defaults.yaml, plus 7 allowlist-feature PRs.

Uses pull_request_target (needed to label fork PRs) without checking out or running PR code; the config is read from the base branch. The action is pinned to the v3.5 commit.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 817c1703-3814-47a6-a9ea-41092ac242af
Copilot AI balanced review requested due to automatic review settings October 12, 2026 10:27
@v-robaiken
v-robaiken requested a review from a team as a code owner October 12, 2026 10:27

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟡 Changes recommended

Reverting the allowlist change leaves a stale label because the path-filtered synchronization workflow will not run.

1 open finding
What changed in this PR

Adds a GitHub Actions workflow to label allowlist-related pull requests.

Changes:

  • Triggers for allowlist YAML changes.
  • Applies the existing allowlist label with restricted permissions.
File Description
.github/​workflows/​label-allowlist-prs.yml Adds automatic allowlist PR labeling.

🧠 Review effort: Balanced


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread .github/workflows/label-allowlist-prs.yml Outdated
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 817c1703-3814-47a6-a9ea-41092ac242af
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 817c1703-3814-47a6-a9ea-41092ac242af
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 817c1703-3814-47a6-a9ea-41092ac242af
@v-robaiken
v-robaiken merged commit 37016ff into main Oct 12, 2026
112 checks passed
@v-robaiken
v-robaiken deleted the robaiken/label-allowlist-prs branch October 12, 2026 12:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants