Skip to content

Build Javascript Bindings in Release Mode - #1865

Merged
spacebear21 merged 3 commits into
payjoin:masterfrom
xstoicunicornx:js-release-mode
Sep 9, 2026
Merged

Build Javascript Bindings in Release Mode#1865
spacebear21 merged 3 commits into
payjoin:masterfrom
xstoicunicornx:js-release-mode

Conversation

@xstoicunicornx

Copy link
Copy Markdown
Collaborator

Updates the Javascript bindings build to use release mode rather than debug mode.

Also includes a small fix up to remove a redundant command for building the Javascript bindings in CONTRIBUTING.md.

Pull Request Checklist

Please confirm the following before requesting review:

@coveralls

coveralls commented Sep 7, 2026

Copy link
Copy Markdown
Collaborator

Coverage Report for CI Build 34275112332

Coverage increased (+0.03%) to 86.64%

Details

  • Coverage increased (+0.03%) from 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: 16474
Covered Lines: 14273
Line Coverage: 86.64%
Coverage Strength: 343.18 hits per line

💛 - Coveralls

spacebear21
spacebear21 previously approved these changes Sep 8, 2026

@spacebear21 spacebear21 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

cACK - I wonder if it's worth making release builds toggle-able or separate commands, so that dev mode can still be used for quicker debugging/tests/CI?

@xstoicunicornx

Copy link
Copy Markdown
Collaborator Author

Updated with commands for debug builds.

@spacebear21 spacebear21 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

ACK

@spacebear21
spacebear21 merged commit 3177747 into payjoin:master Sep 9, 2026
55 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.

3 participants