Skip to content

fix: quote npm dist-tag expression - #53

Merged
coder13 merged 1 commit into
masterfrom
fix/release-dist-tag-quoting
Sep 30, 2026
Merged

coder13 merged 1 commit into
masterfrom
fix/release-dist-tag-quoting

Conversation

@coder13

@coder13 coder13 commented Sep 30, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Correct the Bash quoting in the npm dist-tag calculation.
  • Preserve latest for stable releases and the prerelease name for prereleases.

Tests

  • bash -n passes for the workflow script.
  • Version 1.1.8 produces tag=latest.
  • Version 1.2.0-beta.3 produces tag=beta.

This change prevents the parser failure from release job 109907949210.

Parse the Node expression as one shell argument so release publishing can select the npm tag.
@coder13
coder13 merged commit 0e70f69 into master Sep 30, 2026
2 checks passed
@coveralls

Copy link
Copy Markdown

Coverage Report for CI Build 36723190406

Coverage remained the same at 97.603%

Details

  • Coverage remained the same as the base build.
  • Patch coverage: No coverable lines changed in this PR.
  • No coverage regressions found.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 154
Covered Lines: 153
Line Coverage: 99.35%
Relevant Branches: 138
Covered Branches: 132
Branch Coverage: 95.65%
Branches in Coverage %: Yes
Coverage Strength: 31.04 hits per line

💛 - Coveralls

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