Skip to content

fix: build before test in prepublishOnly#35

Merged
rohal12 merged 1 commit intomainfrom
fix/prepublish-order
Mar 8, 2026
Merged

fix: build before test in prepublishOnly#35
rohal12 merged 1 commit intomainfrom
fix/prepublish-order

Conversation

@rohal12
Copy link
Owner

@rohal12 rohal12 commented Mar 8, 2026

Summary

  • Swaps prepublishOnly script order from test && build to build && test
  • Fixes npm release failure where semantic-release bumps package.json version but dist/format.js still has the old version when tests run

Context

Failed run: https://github.com/rohal12/spindle/actions/runs/22821801248

release-npm

🤖 Generated with Claude Code

When semantic-release bumps package.json, tests ran against stale
dist/format.js with the old version. Building first ensures the
version in dist/format.js matches before the test validates it.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@github-actions
Copy link

github-actions bot commented Mar 8, 2026

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 74.55% 1855 / 2488
🔵 Statements 73.75% 2035 / 2759
🔵 Functions 68.43% 349 / 510
🔵 Branches 65% 938 / 1443
File CoverageNo changed files found.
Generated in workflow #93 for commit a948a22 by the Vitest Coverage Report Action

@rohal12 rohal12 merged commit e948a44 into main Mar 8, 2026
5 checks passed
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