Skip to content

Pack instead of publish for the CI dry-run - #6

Closed
DominicFrei wants to merge 1 commit into
mainfrom
df/ci-pack-dry-run
Closed

Pack instead of publish for the CI dry-run#6
DominicFrei wants to merge 1 commit into
mainfrom
df/ci-pack-dry-run

Conversation

@DominicFrei

@DominicFrei DominicFrei commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

Refs GAF-668

The Node 24 CI jobs fail on every pull request since 0.0.2 went to npm: newer npm rejects npm publish --dry-run when package.json holds an already released version, and between releases that is always the case.

  • CI now runs npm pack --dry-run, which proves the same packaging (same pack machinery, same file list) without the registry version check.
  • The release workflow still runs a real npm publish, so release-time validation is unchanged.
  • This also clears the red CI on PR Add install and uninstall commands #5.

npm publish --dry-run fails on newer npm whenever package.json holds an
already released version, which is the normal state of every pull request
between releases, so the Node 24 jobs went red on any change once 0.0.2 was
published. npm pack --dry-run proves the same packaging and skips only the
registry version check, which the real publish in the release workflow still
enforces.
@DominicFrei

Copy link
Copy Markdown
Contributor Author

Folding this into #5, which now carries the same npm pack --dry-run change, so #5's Node 24 CI goes green on its own without waiting on this to merge first. Closing to keep it to one PR. Fix is unchanged.

@DominicFrei
DominicFrei deleted the df/ci-pack-dry-run branch August 18, 2026 06:55
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