Skip to content

fix(check-merge-safety): Keep failure description short on comment matches, add target_url to failure statuses#781

Merged
avwu99 merged 4 commits intomainfrom
auwu/fix-description
Apr 23, 2026
Merged

fix(check-merge-safety): Keep failure description short on comment matches, add target_url to failure statuses#781
avwu99 merged 4 commits intomainfrom
auwu/fix-description

Conversation

@avwu99
Copy link
Copy Markdown
Contributor

@avwu99 avwu99 commented Apr 23, 2026

📝 Description

  1. Added getWorkflowRunUrl() helper - builds the URL to the current workflow run using githubContext.serverUrl, repo, and runId
  2. Added target_url to failure states - for comment paths, global files, and project failures, the commit status now includes a link to the workflow run where full details are logged
  3. Simplified comment paths failure message - changed from listing paths (which could exceed 140 char limit) to Branch is behind on N path(s) from comment. Please update with main.

Now when a merge safety check fails, users can click the "Details" link next to the status check to go directly to the workflow logs where the full list of outdated paths is printed via core.error().

🔗 Related Issues

@avwu99 avwu99 requested a review from a team as a code owner April 23, 2026 15:51
Comment thread src/helpers/check-merge-safety.ts Outdated
state,
context,
description: message,
target_url,
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.

Suggested change
target_url,
target_url: getWorkflowRunUrl(),

Can't we just inline it here?

@avwu99 avwu99 merged commit 5a4c24c into main Apr 23, 2026
10 checks passed
@avwu99 avwu99 deleted the auwu/fix-description branch April 23, 2026 16:22
@eg-oss-ci
Copy link
Copy Markdown

🎉 This PR is included in version 1.87.2 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants