Skip to content

build: Pin Payjoin release version in source - #18

Merged
ValeraFinebits merged 2 commits into
feat/async-payjoinfrom
build/Pin-Payjoin-release-version-in-source
Oct 23, 2026
Merged

ValeraFinebits merged 2 commits into
feat/async-payjoinfrom
build/Pin-Payjoin-release-version-in-source

Conversation

@ValeraFinebits

Copy link
Copy Markdown
Owner

No description provided.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟢 Approval recommended

Static inspection supports consistent version and build-suffix handling, with no blocking issues identified; the release workflow was not executed.

No open issues 🎉

What changed in this PR

This PR pins the Payjoin release version in source and uses it consistently in build metadata and release archives, while preserving Bitcoin Core’s RPC and peer version identifiers.

Changes:

  • Adds a shared Payjoin version definition and passes it to the client-version build target.
  • Includes the Payjoin release version in full version strings.
  • Derives archive names from source and verifies packaged binary versions and isolated regtest RPC output.
File Description
src/​payjoin/​version.cmake Defines the pinned Payjoin release version.
src/​payjoin/​CMakeLists.txt Supplies the version to bitcoin_clientversion.
src/​clientversion.cpp Adds Payjoin metadata while retaining build suffix handling.
.github/​workflows/​payjoin-release.yml Reads source versions and checks archive, binary, and RPC consistency.

Review effort: Balanced

@ValeraFinebits
ValeraFinebits marked this pull request as ready for review October 23, 2026 08:13
@ValeraFinebits
ValeraFinebits merged commit 4945fc2 into feat/async-payjoin Oct 23, 2026
57 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.

2 participants