Skip to content

fix: remove edited type trigger from enforce-release-source ci#2896

Open
trueberryless wants to merge 2 commits into
npmx-dev:mainfrom
trueberryless:chore/fix-ci
Open

fix: remove edited type trigger from enforce-release-source ci#2896
trueberryless wants to merge 2 commits into
npmx-dev:mainfrom
trueberryless:chore/fix-ci

Conversation

@trueberryless

@trueberryless trueberryless commented Jun 12, 2026

Copy link
Copy Markdown
Member

🧭 Context

CI on main is (seemingly randomly) failing because the enforce-release-source action gets cancelled. This PR aims to fix this issue.

📚 Description

To my understanding, the problem is cause because merge queues quickly add commits to main, which causes two things:

  1. the commit from main lands in the automatically generated next release PR, which causes the enforce-release-source action to trigger because of the synchronize type
  2. the release-pr workflow updates the PR description, which causes the enforce-release-source action to trigger a second time within a short time frame because of the edited type

This results in two workflow runs getting triggered and one getting cancelled because both events trigger quickly after each other. Therefore, removing the edited type should resolve the issue, as it removes one of the concurrent runs. Since an edit of the PR description does not include target branch changes, removing the edited type would be my solution. I also explicitly set cancel-in-progress to false to avoid future confusion.

So I think this cancellation happens seemingly random because workflows just do not take the same time always, resulting in the cancellation sometimes happening because of timing reasons, and sometimes not.

This is just how I understood it after debugging for the last 2 hours. I would really appreciate another four eyes on my deduction.

@trueberryless trueberryless requested a review from alexdln June 12, 2026 21:22
@vercel

vercel Bot commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
npmx.dev Ready Ready Preview, Comment Jun 13, 2026 10:22am
2 Skipped Deployments
Project Deployment Actions Updated (UTC)
docs.npmx.dev Ignored Ignored Preview Jun 13, 2026 10:22am
npmx-lunaria Ignored Ignored Jun 13, 2026 10:22am

Request Review

@coderabbitai

coderabbitai Bot commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Walkthrough

This PR updates the enforce-release-source GitHub Actions workflow to trigger on PR commit synchronizations (synchronize) instead of PR edits, and it adds cancel-in-progress: false to the workflow concurrency configuration.

Changes

Workflow configuration updates

Layer / File(s) Summary
Trigger event types
.github/workflows/enforce-release-source.yml
Replaced pull_request_target.types entry edited with synchronize so the workflow runs on commits pushed to a PR.
Concurrency configuration
.github/workflows/enforce-release-source.yml
Added cancel-in-progress: false beneath the existing concurrency.group setting to prevent automatic cancellation of in-progress runs.

Suggested reviewers

  • MatteoGabriele

Important

Pre-merge checks failed

Please resolve all errors before merging. Addressing warnings is optional.

❌ Failed checks (1 error)

Check name Status Explanation Resolution
Title check ❌ Error The title states 'remove edited type trigger' but the PR objectives describe removing both the edited trigger AND the concurrency block, with the concurrency block being the primary fix. Update the title to accurately reflect both key changes: 'fix: remove concurrency block and edited event trigger from enforce-release-source ci' or clarify if only the edited trigger removal is the main concern.
✅ Passed checks (3 passed)
Check name Status Explanation
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description check ✅ Passed The pull request description clearly explains the context, identifies the root cause of CI failures, and details the specific changes being made to resolve the issue.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@codecov

codecov Bot commented Jun 12, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

@trueberryless

Copy link
Copy Markdown
Member Author

@trueberryless trueberryless changed the title fix: remove concurrency block and edit type trigger from enforce-release-source ci fix: remove edited type trigger from enforce-release-source ci Jun 13, 2026
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.

1 participant