Skip to content

fix(misc): Replace abandoned merge-conflict label action - #2784

Closed
bakerboy448 wants to merge 1 commit into
masterfrom
fix/label-conflicts-action
Closed

fix(misc): Replace abandoned merge-conflict label action#2784
bakerboy448 wants to merge 1 commit into
masterfrom
fix/label-conflicts-action

Conversation

@bakerboy448

@bakerboy448 bakerboy448 commented Jun 21, 2026

Copy link
Copy Markdown
Contributor

Problem

The triage job (.github/workflows/conflicts.yml) fails on every push and PR. mschilde/auto-label-merge-conflicts builds a Docker image whose Dockerfile runs yarn install --frozen-lockfile on node:alpine, which no longer ships yarn:

/bin/sh: yarn: not found
ERROR: process "/bin/sh -c yarn install --frozen-lockfile" did not complete successfully: exit code: 127

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:

old new
CONFLICT_LABEL_NAME dirtyLabel
GITHUB_TOKEN repoToken
MAX_RETRIES retryMax
WAIT_MS: 5000 (ms) retryAfter: 5 (sec)

Also added continueOnMissingPermissions: true so PRs from forks (which only get a read-only token) don't fail the check.

Pinned to SHA e0e7fd9 with a version comment, matching the repo's pinning convention.

Summary by Sourcery

CI:

  • Update the merge-conflict triage workflow to use eps1lon/actions-label-merge-conflict with equivalent settings and support for missing permissions on forked PRs.

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.
@sourcery-ai

sourcery-ai Bot commented Jun 21, 2026

Copy link
Copy Markdown
Contributor
Reviewer's guide (collapsed on small PRs)

Reviewer's Guide

Replaces 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 Action

flowchart 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
Loading

File-Level Changes

Change Details Files
Swap out the unmaintained Docker-based merge-conflict labeling action for a maintained Node-based one and remap its configuration. .github/workflows/conflicts.yml

Tips and commands

Interacting with Sourcery

  • Trigger a new review: Comment @sourcery-ai review on the pull request.
  • Continue discussions: Reply directly to Sourcery's review comments.
  • Generate a GitHub issue from a review comment: Ask Sourcery to create an
    issue from a review comment by replying to it. You can also reply to a
    review comment with @sourcery-ai issue to create an issue from it.
  • Generate a pull request title: Write @sourcery-ai anywhere in the pull
    request title to generate a title at any time. You can also comment
    @sourcery-ai title on the pull request to (re-)generate the title at any time.
  • Generate a pull request summary: Write @sourcery-ai summary anywhere in
    the pull request body to generate a PR summary at any time exactly where you
    want it. You can also comment @sourcery-ai summary on the pull request to
    (re-)generate the summary at any time.
  • Generate reviewer's guide: Comment @sourcery-ai guide on the pull
    request to (re-)generate the reviewer's guide at any time.
  • Resolve all Sourcery comments: Comment @sourcery-ai resolve on the
    pull request to resolve all Sourcery comments. Useful if you've already
    addressed all the comments and don't want to see them anymore.
  • Dismiss all Sourcery reviews: Comment @sourcery-ai dismiss on the pull
    request to dismiss all existing Sourcery reviews. Especially useful if you
    want to start fresh with a new review - don't forget to comment
    @sourcery-ai review to trigger a new review!

Customizing Your Experience

Access your dashboard to:

  • Enable or disable review features such as the Sourcery-generated pull request
    summary, the reviewer's guide, and others.
  • Change the review language.
  • Add, remove or edit custom review instructions.
  • Adjust other review settings.

Getting Help

@github-actions github-actions Bot added the Area: github_actions Github Actions Related label Jun 21, 2026

@sourcery-ai sourcery-ai Bot 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.

Hey - I've reviewed your changes and they look great!


Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

@bakerboy448

Copy link
Copy Markdown
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 pull_request_target (write-capable token, with a 'never checks out PR code' safety note), which is a cleaner fix than the continueOnMissingPermissions approach here. No changes needed.

@bakerboy448
bakerboy448 deleted the fix/label-conflicts-action branch June 21, 2026 04:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: github_actions Github Actions Related

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant