Skip to content

ci: skip chromatic on forks#1685

Closed
rzzf wants to merge 1 commit intonpmx-dev:mainfrom
rzzf:ci/skip-fork
Closed

ci: skip chromatic on forks#1685
rzzf wants to merge 1 commit intonpmx-dev:mainfrom
rzzf:ci/skip-fork

Conversation

@rzzf
Copy link
Contributor

@rzzf rzzf commented Feb 27, 2026

🔗 Linked issue

resolves #1683

🧭 Context

To prevent error emails caused by workflow failures when the repository is forked.

📚 Description

@vercel
Copy link

vercel bot commented Feb 27, 2026

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

Project Deployment Actions Updated (UTC)
npmx.dev Ready Ready Preview, Comment Feb 27, 2026 5:15am
2 Skipped Deployments
Project Deployment Actions Updated (UTC)
docs.npmx.dev Ignored Ignored Preview Feb 27, 2026 5:15am
npmx-lunaria Ignored Ignored Feb 27, 2026 5:15am

Request Review

@codecov
Copy link

codecov bot commented Feb 27, 2026

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!

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 27, 2026

📝 Walkthrough

Walkthrough

The Chromatic workflow configuration has been updated to add a repository-level guard condition. The workflow job now includes an additional check to ensure it executes only when the repository is exactly "npmx-dev/npmx.dev". The existing logic that handles pull request events—either running on non-pull-request events or when the pull request head repository matches the target repository—is preserved. This change restricts the Chromatic job execution scope whilst maintaining the previous event-type handling behaviour.

Suggested reviewers

  • danielroe
🚥 Pre-merge checks | ✅ 1
✅ Passed checks (1 passed)
Check name Status Explanation
Description check ✅ Passed The pull request description is directly related to the changeset, explaining the purpose of adding a repository guard to the Chromatic workflow to prevent errors on forked repositories.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

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.

@rzzf
Copy link
Contributor Author

rzzf commented Feb 27, 2026

fixed by #1679

@rzzf rzzf closed this Feb 27, 2026
@rzzf rzzf deleted the ci/skip-fork branch February 27, 2026 08:33
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.

Chromatic Action runs on Forks

1 participant