Skip to content

Removing the latent labels from the old branching setup#2130

Merged
aaronpowell merged 1 commit into
mainfrom
workflow-cleanup
Jun 26, 2026
Merged

Removing the latent labels from the old branching setup#2130
aaronpowell merged 1 commit into
mainfrom
workflow-cleanup

Conversation

@aaronpowell

Copy link
Copy Markdown
Contributor

Removing some missed legacy labelling

Copilot AI review requested due to automatic review settings June 26, 2026 00:10
@github-actions github-actions Bot added targets-main PR targets main instead of staged workflow PR touches workflow automation labels Jun 26, 2026
GitHub Advanced Security started work on behalf of aaronpowell June 26, 2026 00:10 View session
GitHub Advanced Security finished work on behalf of aaronpowell June 26, 2026 00:11

Copilot AI left a comment

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.

Pull request overview

This PR removes two legacy PR-intent labels from the repository’s label management workflows, aligning labeling automation with the current (non-legacy) branching/labeling model.

Changes:

  • Removes the targets-main and branched-main label definitions from the repository label setup workflow.
  • Removes targets-main/branched-main from PR intent labeling logic, including the old path-based detection for branched-main.
  • Simplifies the PR labeling flow to apply category labels directly based on file-path patterns.
Show a summary per file
File Description
.github/workflows/setup-labels.yml Stops creating/updating the legacy targets-main and branched-main labels.
.github/workflows/label-pr-intent.yml Removes legacy label management and the special-case branched-main detection logic from PR labeling.

Review details

Comments suppressed due to low confidence (1)

.github/workflows/label-pr-intent.yml:26

  • Removing targets-main / branched-main from managedLabels means the workflow will no longer remove these legacy labels from existing PRs, so they can linger indefinitely even though the repo has stopped applying them. If the goal is to fully retire these labels, keep them in managedLabels (but do not add them to desiredLabels) so the workflow automatically cleans them up over time.
            const managedLabels = {
              'skills': true,
              'plugin': true,
              'agent': true,
              'instructions': true,
  • Files reviewed: 2/2 changed files
  • Comments generated: 0
  • Review effort level: Low

@aaronpowell aaronpowell merged commit 4c852c8 into main Jun 26, 2026
19 checks passed
@aaronpowell aaronpowell deleted the workflow-cleanup branch June 26, 2026 00:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

targets-main PR targets main instead of staged workflow PR touches workflow automation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants