Skip to content

ci(deploy-site): run test and build in parallel#149

Merged
jaredwray merged 1 commit into
mainfrom
claude/parallel-test-build-deploy-Dlflx
May 18, 2026
Merged

ci(deploy-site): run test and build in parallel#149
jaredwray merged 1 commit into
mainfrom
claude/parallel-test-build-deploy-Dlflx

Conversation

@jaredwray

Copy link
Copy Markdown
Owner

Summary

  • build no longer waits on test — they now run in parallel.
  • deploy now requires both test and build to succeed (needs: [test, build]), so the test gate is preserved while wall-clock time on a release drops to roughly max(test, build) + deploy.
  • build never consumed any output from test, so this change is purely a scheduling optimization.

Job graph

test ─┐
      ├──> deploy
build ┘

Test plan

  • Trigger deploy-site via workflow_dispatch and confirm test and build start in parallel and deploy waits for both.
  • On a throwaway commit, force a failing test and confirm deploy is skipped.
  • On a green run, confirm the dist artifact is uploaded by build and downloaded by deploy as before.

Generated by Claude Code

The build job did not consume any output from test, so chaining them
sequentially only added wall-clock time. Drop the needs: test from
build and make deploy require both test and build to succeed, so a
release ships faster without losing the test gate.
@gemini-code-assist

Copy link
Copy Markdown

Note

Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported.

@codecov

codecov Bot commented May 18, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (7e3e302) to head (133115d).

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #149   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           37        37           
  Lines         1089      1089           
  Branches       222       222           
=========================================
  Hits          1089      1089           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@jaredwray jaredwray merged commit 6524834 into main May 18, 2026
9 checks passed
@jaredwray jaredwray deleted the claude/parallel-test-build-deploy-Dlflx branch May 18, 2026 20:43
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.

2 participants