Skip to content

fix: skip npm publish if boa_wasm version already exists#4654

Merged
jedel1043 merged 5 commits intoboa-dev:mainfrom
JkrishnaD:fix/npm-wasm-publish
Feb 25, 2026
Merged

fix: skip npm publish if boa_wasm version already exists#4654
jedel1043 merged 5 commits intoboa-dev:mainfrom
JkrishnaD:fix/npm-wasm-publish

Conversation

@JkrishnaD
Copy link
Contributor

This Pull Request fixes/closes #4065 .

It changes the following:

  • Added npm version check using npm view @boa-dev/boa_wasm@$VERSION
  • Skip npm publish step when the version already exists

@github-actions
Copy link

github-actions bot commented Feb 21, 2026

Test262 conformance changes

Test result main count PR count difference
Total 52,862 52,862 0
Passed 49,497 49,497 0
Ignored 2,255 2,255 0
Failed 1,110 1,110 0
Panics 0 0 0
Conformance 93.63% 93.63% 0.00%

@jedel1043 jedel1043 added webassembly Anything related to using Boa with WASM github_actions Pull requests that update Github_actions code waiting-on-author Waiting on PR changes from the author labels Feb 22, 2026
@jedel1043 jedel1043 added this to the v1.0.0 milestone Feb 22, 2026
JkrishnaD and others added 2 commits February 22, 2026 23:33
Co-authored-by: José Julián Espina <jedel0124@gmail.com>
@codecov
Copy link

codecov bot commented Feb 22, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 56.62%. Comparing base (6ddc2b4) to head (fe290c4).
⚠️ Report is 679 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4654      +/-   ##
==========================================
+ Coverage   47.24%   56.62%   +9.37%     
==========================================
  Files         476      548      +72     
  Lines       46892    60083   +13191     
==========================================
+ Hits        22154    34021   +11867     
- Misses      24738    26062    +1324     

☔ 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.

Copy link
Member

@jedel1043 jedel1043 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice, thank you for the fix!

@jedel1043 jedel1043 removed the waiting-on-author Waiting on PR changes from the author label Feb 25, 2026
@jedel1043 jedel1043 added this pull request to the merge queue Feb 25, 2026
Merged via the queue into boa-dev:main with commit c89acb6 Feb 25, 2026
18 checks passed
@JkrishnaD JkrishnaD deleted the fix/npm-wasm-publish branch February 25, 2026 07:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

github_actions Pull requests that update Github_actions code webassembly Anything related to using Boa with WASM

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Don't publish boa_wasm to npm if version exists

2 participants