fix(misc): Replace abandoned merge-conflict label action - #2784
Closed
bakerboy448 wants to merge 1 commit into
Closed
fix(misc): Replace abandoned merge-conflict label action#2784bakerboy448 wants to merge 1 commit into
bakerboy448 wants to merge 1 commit into
Conversation
The 'triage' job failed on every push/PR: mschilde/auto-label-merge-conflicts
builds a Docker image that runs 'yarn install' on node:alpine, which no longer
ships yarn ('yarn: not found', exit 127), so the action never builds.
Replace it with the maintained eps1lon/actions-label-merge-conflict@v3.1.0
(node24 JS action — no Docker/yarn). Input mapping:
CONFLICT_LABEL_NAME->dirtyLabel, GITHUB_TOKEN->repoToken, MAX_RETRIES->retryMax,
WAIT_MS:5000(ms)->retryAfter:5(sec). Also adds continueOnMissingPermissions so
PRs from forks (read-only token) don't fail the check.
Contributor
Reviewer's guide (collapsed on small PRs)Reviewer's GuideReplaces the failing, abandoned merge-conflict labeling GitHub Action with a maintained alternative and updates workflow inputs and permissions handling accordingly. Flow diagram for the updated merge-conflict labeling GitHub Actionflowchart TD
GitHubEvent[GitHub push/PR event]
Workflow[triage workflow conflicts.yml]
Step[Label merge conflicts\n eps1lon/actions-label-merge-conflict]
Perm{Has write permissions?}
Conflict{Merge conflicts detected?}
Label[Apply dirtyLabel\n Status: Conflicted]
Success[Job succeeds]
GitHubEvent --> Workflow
Workflow --> Step
Step --> Perm
Perm -- Yes --> Conflict
Perm -- No --> Success
Conflict -- Yes --> Label
Conflict -- No --> Success
Label --> Success
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
Contributor
Author
|
Closing as superseded — master already fixed this via #2748 (switch to eps1lon/actions-label-merge-conflict + fix fork-PR failures) and #2757 (bump to v3.1.0). Master's version additionally moves to |
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.
Problem
The
triagejob (.github/workflows/conflicts.yml) fails on every push and PR.mschilde/auto-label-merge-conflictsbuilds a Docker image whoseDockerfilerunsyarn install --frozen-lockfileonnode:alpine, which no longer ships yarn:The action is unmaintained, so this won't be fixed upstream.
Fix
Replace it with the maintained eps1lon/actions-label-merge-conflict@v3.1.0 (a node24 JS action — no Docker, no yarn). Inputs mapped 1:1:
CONFLICT_LABEL_NAMEdirtyLabelGITHUB_TOKENrepoTokenMAX_RETRIESretryMaxWAIT_MS: 5000(ms)retryAfter: 5(sec)Also added
continueOnMissingPermissions: trueso PRs from forks (which only get a read-only token) don't fail the check.Pinned to SHA
e0e7fd9with a version comment, matching the repo's pinning convention.Summary by Sourcery
CI: