Skip to content

fix(ci): support fork PR preview builds and smoke tests#6428

Closed
rubencarvalho wants to merge 2 commits into
mainfrom
rubencarvalho/fix-fork-pr-preview-ci
Closed

fix(ci): support fork PR preview builds and smoke tests#6428
rubencarvalho wants to merge 2 commits into
mainfrom
rubencarvalho/fix-fork-pr-preview-ci

Conversation

@rubencarvalho

Copy link
Copy Markdown
Contributor

Summary

  • Skip Azure Blob Storage deploy on fork PRs (and when simulating with the test_fork_preview label) because repository secrets are unavailable to external contributions
  • Keep documentation/Storybook builds green and run Playwright smoke tests against locally served build artifacts for fork PRs
  • Gracefully skip Chromatic on fork PRs that have the run_vrt label instead of failing on an empty CHROMATIC_TOKEN

Test plan

  • Add test_fork_preview label to this PR and confirm preview workflow skips Azure deploy, posts fork-style comment, and smoke tests pass
  • Remove test_fork_preview label and confirm normal Azure preview links are deployed
  • Verify a real fork PR (e.g. fix(menu): Use package import for sp-menu-divider registration #6420) passes build + smoke tests after merge

Made with Cursor

Skip Azure and Chromatic steps that need secrets on fork PRs.
Documentation builds and Playwright smoke tests still run via
locally served build artifacts.

Co-authored-by: Cursor <cursoragent@cursor.com>
@rubencarvalho rubencarvalho requested a review from a team as a code owner June 19, 2026 11:23
@changeset-bot

changeset-bot Bot commented Jun 19, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 85d38c6

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@github-actions

github-actions Bot commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

📚 Branch Preview Links

✅ Documentation and Storybook builds completed successfully.

Preview deployment is not available for pull requests from forked repositories. GitHub does not expose repository secrets to workflows triggered by external contributions, so Azure Blob Storage uploads are skipped. Smoke tests still run against the built documentation served locally in CI.

This PR is using the test_fork_preview label to simulate fork PR behavior on an internal branch.

@coveralls

coveralls commented Jun 19, 2026

Copy link
Copy Markdown
Collaborator

Coverage Report for CI Build 27823134133

Warning

No base build found for commit 8551787 on main.
Coverage changes can't be calculated without a base build.
If a base build is processing, this comment will update automatically when it completes.

Coverage: 96.24%

Details

  • Patch coverage: No coverable lines changed in this PR.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

Requires a base build to compare against. How to fix this →


Coverage Stats

Coverage Status
Relevant Lines: 39108
Covered Lines: 37837
Line Coverage: 96.75%
Relevant Branches: 6450
Covered Branches: 6008
Branch Coverage: 93.15%
Branches in Coverage %: Yes
Coverage Strength: 458.45 hits per line

💛 - Coveralls

@rubencarvalho rubencarvalho marked this pull request as draft June 19, 2026 12:30
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.

2 participants