Skip to content

Make checks workflow label-driven via run-type output#8524

Merged
ludeeus merged 5 commits into
masterfrom
claude/brave-darwin-mlwl9r
Jun 15, 2026
Merged

Make checks workflow label-driven via run-type output#8524
ludeeus merged 5 commits into
masterfrom
claude/brave-darwin-mlwl9r

Conversation

@ludeeus

@ludeeus ludeeus commented Jun 15, 2026

Copy link
Copy Markdown
Member

Replace the fragile 'removal' detection (which inferred removals from a 'Bad data []' parse-error sentinel) with an explicit, label-driven run-type output on the preflight job:

  • remove-repositories -> removal
  • New default repository -> new-repository
  • renamed-repositories -> rename
  • no known label -> unknown

Replace the fragile 'removal' detection (which inferred removals from a
'Bad data []' parse-error sentinel) with an explicit, label-driven
run-type output on the preflight job:

- remove-repositories   -> removal
- New default repository -> new-repository
- renamed-repositories  -> renamed
- no known label        -> unknown

Fail the preflight if more than one of these labels is set. Only
new-repository PRs run the preflight detection steps and the downstream
jobs (except editable). Add the labeled trigger so applying a label
re-runs the checks.
Comment thread .github/workflows/checks.yml Outdated

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.

Pull request overview

This PR updates the checks GitHub Actions workflow to determine what validation to run based on PR labels, replacing the prior “removal” inference logic with an explicit preflight “run type” output and enabling re-runs when labels change.

Changes:

  • Add pull_request triggers for labeled/unlabeled so applying/removing labels re-runs checks.
  • Introduce a preflight “run type” decision based on mutually-exclusive labels (remove-repositories, New default repository, renamed-repositories).
  • Gate the downstream validation jobs so they run only for new-repository PRs (except the editable job).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/checks.yml
Comment thread .github/workflows/checks.yml Outdated
Comment thread .github/workflows/checks.yml Outdated
Comment thread .github/workflows/checks.yml Outdated
Comment thread .github/workflows/checks.yml Outdated
Comment thread .github/workflows/checks.yml Outdated
Comment thread .github/workflows/checks.yml Outdated
Comment thread .github/workflows/checks.yml Outdated
Comment thread .github/workflows/checks.yml Outdated
Comment thread .github/workflows/checks.yml Outdated

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.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

Comment thread .github/workflows/checks.yml

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.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 2 comments.

Comment thread .github/workflows/checks.yml
Comment thread .github/workflows/checks.yml
@ludeeus ludeeus merged commit 2fd343d into master Jun 15, 2026
15 checks passed
@ludeeus ludeeus deleted the claude/brave-darwin-mlwl9r branch June 15, 2026 14:18
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