Skip to content

Bump payjoin version to 1.2.0 - #1931

Merged
DanGould merged 2 commits into
masterfrom
bump-payjoin-1-2-0
Oct 2, 2026
Merged

DanGould merged 2 commits into
masterfrom
bump-payjoin-1-2-0

Conversation

@benalleng

@benalleng benalleng commented Sep 30, 2026 •

Copy link
Copy Markdown
Collaborator

This bumps payjoin to 1.2.0. This is a minor release instead of a patch because of the addition of the new API methods made available in the non-blocking interface.

The first commit fixes a Kotlin test failure following the #1733 BuildReceiverError rename.

Changelog written by GLM 5.3

Pull Request Checklist

Please confirm the following before requesting review:

@benalleng
benalleng added this pull request to stack #1933 September 30, 2026 16:50
@benalleng benalleng changed the title Bump payjoin 1 2 0 Bump payjoin version to 1.2.0 Sep 30, 2026
@benalleng benalleng mentioned this pull request Sep 30, 2026
3 tasks done
@coveralls

coveralls commented Sep 30, 2026 •

Copy link
Copy Markdown
Collaborator

Coverage Report for CI Build 36981169913

Coverage remained the same at 87.289%

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: 17804
Covered Lines: 15541
Line Coverage: 87.29%
Coverage Strength: 320.51 hits per line

💛 - Coveralls

@benalleng
benalleng force-pushed the bump-payjoin-1-2-0 branch 2 times, most recently from 6e67483 to 083dd19 Compare October 1, 2026 13:01
This is a minor release as it adds the non-blocking interface and the
related methods that can now be used instead of synchronous callbacks.
This all keeps the existing methods so it remains backwards compatible.
@DanGould

DanGould commented Oct 2, 2026

Copy link
Copy Markdown
Member

The first commit fixes a Kotlin test failure following the #1733 BuildReceiverError rename.

How did you find the error and why was it not caught by CI? no CI for kotlin?

@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

@DanGould
DanGould merged commit 8e07297 into master Oct 2, 2026
78 checks passed
@benalleng

Copy link
Copy Markdown
Collaborator Author

How did you find the error and why was it not caught by CI? no CI for kotlin?

CI did fail here. I think this may have gotten through as #1733 might not have been rebased properly following the update to the expanded kotlin tests. So it showed green but only because most kotlin tests at that point were no-ops or incomplete

It's likely master CI failed post merge I jus didn't catch it

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.

4 participants