Skip to content

chore!: bump MSRV to 1.75 - #31

Merged
noahjoeris merged 1 commit into
bitcoindevkit:masterfrom
noahjoeris:chore/msrv-1.75
Oct 26, 2026
Merged

noahjoeris merged 1 commit into
bitcoindevkit:masterfrom
noahjoeris:chore/msrv-1.75

Conversation

@noahjoeris

@noahjoeris noahjoeris commented Oct 20, 2026 •

Copy link
Copy Markdown
Contributor

Description

Bump MSRV from 1.70 to 1.75 to match electrum-client (1.75).

Changelog notice

- Breaking: Raise MSRV to 1.75.

🤖 Generated with Claude Code

@noahjoeris noahjoeris self-assigned this Oct 20, 2026
@noahjoeris noahjoeris added this to the 0.5.0 milestone Oct 20, 2026
@noahjoeris noahjoeris added the api A breaking API change label Oct 20, 2026

@evanlinjin evanlinjin left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

ACK ec66da7

@evanlinjin evanlinjin left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Btw, the commit message is wrong:

Match the MSRV of electrum-client and LDK's lightning-transaction-sync so LDK can adopt this crate
without raising its own MSRV.

We are raising the MSRV here so it wouldn't help at all.

@noahjoeris

Copy link
Copy Markdown
Contributor Author

Yes thanks, will update. Claude got out of control 🤖

@evanlinjin evanlinjin left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

ACK 3467d88

@codecov

codecov Bot commented Oct 25, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 50.00000% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 48.33%. Comparing base (8f480f9) to head (dc49500).

Files with missing lines Patch % Lines
src/client.rs 50.00% 1 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##           master      #31   +/-   ##
=======================================
  Coverage   48.33%   48.33%           
=======================================
  Files          11       11           
  Lines         902      902           
  Branches       18       18           
=======================================
  Hits          436      436           
  Misses        461      461           
  Partials        5        5           
Flag Coverage Δ
rust 48.33% <50.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Match electrum-client's MSRV. Use `io::Error::other`, which Clippy
requires once the MSRV is 1.74 or higher.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

@evanlinjin evanlinjin left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

ACK dc49500

@noahjoeris
noahjoeris merged commit b94748c into bitcoindevkit:master Oct 26, 2026
15 of 17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api A breaking API change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants