Skip to content

build(silentpayments): raise MSRV to 1.85.0 - #71

Merged
nymius merged 1 commit into
bitcoindevkit:masterfrom
kkdao:fix/msrv-core-error
Aug 26, 2026
Merged

nymius merged 1 commit into
bitcoindevkit:masterfrom
kkdao:fix/msrv-core-error

Conversation

@kkdao

@kkdao kkdao commented Aug 21, 2026 •

Copy link
Copy Markdown
Contributor

Description

Fixes #69

bdk_sp declares Rust 1.63, but the crate uses core::error::Error, which requires Rust 1.81 or newer.

This raises the declared MSRV to Rust 1.85.0, matching current BDK packages, and removes the stale CONTRIBUTING.md claim that Rust 1.63 is enforced by CI.

The implementation and workflow remain unchanged.

Verification

The full workspace checks, Clippy, tests, and documentation tests pass. The bdk_sp library also builds with exact Rust 1.85.0 using all features and the committed lockfile.

Changelog notice

Changed: bdk_sp now requires Rust 1.85.0.

@kkdao
kkdao force-pushed the fix/msrv-core-error branch from 8d9765f to a8523ae Compare August 21, 2026 21:36
@kkdao
kkdao marked this pull request as draft August 21, 2026 21:48
@kkdao
kkdao marked this pull request as ready for review August 21, 2026 21:56
@kkdao kkdao changed the title fix: honour the declared Rust 1.63 MSRV fix: honor the declared Rust 1.63 MSRV Aug 24, 2026
Comment thread CONTRIBUTING.md
@kkdao kkdao changed the title fix: honor the declared Rust 1.63 MSRV build(silentpayments): raise MSRV to 1.85.0 Aug 26, 2026
@kkdao
kkdao force-pushed the fix/msrv-core-error branch from bd9c721 to 659d2fe Compare August 26, 2026 03:50
bdk_sp declared Rust 1.63, but the crate uses core::error::Error, which
requires Rust 1.81 or newer. Raise the declared MSRV to 1.85.0 to match
current BDK packages, and drop the stale CONTRIBUTING.md claim that
1.63.0 is enforced by CI.
@kkdao
kkdao force-pushed the fix/msrv-core-error branch from 01636bb to 3298491 Compare August 26, 2026 03:56

@nymius nymius 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 3298491

@nymius
nymius merged commit f6609f8 into bitcoindevkit:master Aug 26, 2026
@kkdao
kkdao deleted the fix/msrv-core-error branch August 26, 2026 16:46
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.

bdk_sp 0.2.0 does not satisfy its declared Rust 1.63 MSRV

2 participants