chore: replace labeler workflows with Carson's auto-labeler and conflicts-notifier - #10547
Merged
Merged
Conversation
Member
Author
|
The failure is expected, as I opened this from my fork. |
michalsn
approved these changes
Sep 13, 2026
paulbalandan
force-pushed
the
carson-labels
branch
from
September 13, 2026 09:13
6c53f02 to
ad86084
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Second step of the move onto Carson, following #10529 and #10553. This PR retires both label workflows and replaces them with two Carson subscribers configured in
.github/carson.yml.actions/labeler(label-pr.yml+labeler.yml) becomesauto-labeler. The four rules port one-to-one:4.8on base branch,github_actionson workflow changes,documentationandtestingwhen every changed file is under the respective directory.sync-labels: truecarries over assync_labels: true.label-add-conflict-all-pr.yml) becomesconflicts-notifier, with the samestalelabel and the same comment text.They ship together because Carson routes the conflict label through
auto-labeler, so enablingconflicts-notifieralone would post the comment but skip the label.Changes:
.github/carson.yml: enableauto-labelerandconflicts-notifierwith the settings above..github/workflows/run-carson.yml: addlabeledtopull_request_target, anissuestrigger foropened,edited, andlabeled, andpushondevelopand4.*so conflicts are re-checked when a base branch advances. Thelabeledandissuesevents complete the documented trigger set forauto-labelerand stay inert untilissue_rulesorimplied_labelsare configured..github/labeler.yml,.github/workflows/label-pr.yml, and.github/workflows/label-add-conflict-all-pr.yml. The github-script fork guard inlabel-pr.ymlis no longer needed, since Carson reads its config from the default branch only and never checks out PR code.Behaviour changes worth knowing:
stalelabel is now removed automatically once a PR is conflict-free, and the conflict comment is minimized as resolved. The old script only ever added.None of this takes effect on this PR itself. Under
pull_request_targetthe workflow runs from the base branch, and Carson readscarson.ymlfrom the default branch, so the new triggers and subscribers activate on merge. The merge commit's push todevelopruns the first conflict sweep over open PRs.Next:
signed-commitsreplacing the 1Password action, thenwelcomeandtemplate-enforcerfor Mergeable.Checklist: