Skip to content

Bump bindings for payjoin 1.2.0 - #1945

Merged
DanGould merged 4 commits into
masterfrom
bindings-payjoin-1-2-0
Oct 5, 2026
Merged

DanGould merged 4 commits into
masterfrom
bindings-payjoin-1-2-0

Conversation

@DanGould

@DanGould DanGould commented Oct 4, 2026

Copy link
Copy Markdown
Member

These bump each binding language save for Kotlin which still does not have any release mechanism.

Each binding release is a minor release instead of a patch as the BuildReceiverError and BuildSenderError are breaking changes.

GLM 5.3 wrote the changelog

Pull Request Checklist

Please confirm the following before requesting review:

This bump is not backwards compatible to is a minor bump instead of a
patch release due to the `BuildReceiverError` and `BuildSenderError` renaming.
This is a minor release and not a patch because of the breaking change
caused by the `BuildReceiverError` and  `BuildSenderError` renames.
This is a minor release and not a patch because of the breaking change
from the `BuildReceiverError` and `BuildSenderError` renames.
This is a minor release instead of a patch release because there is a
breaking change with the `BuildReceiverError` and `BuildSenderError` renames.
@DanGould
DanGould requested a review from spacebear21 October 4, 2026 10:56
@DanGould DanGould mentioned this pull request Oct 4, 2026
3 tasks done
@coveralls

coveralls commented Oct 4, 2026 •

Copy link
Copy Markdown
Collaborator

Coverage Report for CI Build 37197030790

Coverage remained the same at 87.494%

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: 17848
Covered Lines: 15616
Line Coverage: 87.49%
Coverage Strength: 319.6 hits per line

💛 - Coveralls

@DanGould
DanGould requested review from benalleng and removed request for spacebear21 October 5, 2026 11:04

@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

## Stability

The package is pre-1.0 while the C# API stabilizes; expect breaking changes between 0.x releases. The version's build metadata names the wrapped payjoin core release, so `0.1.0+payjoin-1.0.0` packages payjoin 1.0.0, the first stable payjoin release.
The package is pre-1.0 while the C# API stabilizes; expect breaking changes between 0.x releases. The version's build metadata names the wrapped payjoin core release, so `0.2.0+payjoin-1.2.0` packages payjoin 1.2.0.

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.

This seems unnecessary to update with every release. Maybe Xs and Ys as placeholders?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Yeah, making this deterministic or better yet as you suggest static if it can't be removed is the way.

@DanGould
DanGould merged commit 6c31c67 into master Oct 5, 2026
71 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.

4 participants