Skip to content

ci: Update rbmt + bump dependencies#55

Merged
ValuedMammal merged 2 commits into
bitcoindevkit:masterfrom
ValuedMammal:ci/setup-rbmt
May 12, 2026
Merged

ci: Update rbmt + bump dependencies#55
ValuedMammal merged 2 commits into
bitcoindevkit:masterfrom
ValuedMammal:ci/setup-rbmt

Conversation

@ValuedMammal
Copy link
Copy Markdown
Collaborator

@ValuedMammal ValuedMammal commented May 11, 2026

Description

Updates cargo-rbmt to match the cargo-rbmt-0.2.1 tag, and adds an exclusion for the stable + Cargo-minimal.lock matrix combination (matching the existing msrv + Cargo-recent.lock exclusion). Also bumps jsonrpc to 0.20.0, bitcoind to 0.38.0, and the pinned stable toolchain to 1.95.0.

Changelog notice

Changed

  • Bump cargo-rbmt to 0.2.1
  • Bump jsonrpc to 0.20.0
  • Bump bitcoind to 0.38.0
  • Update stable toolchain to 1.95.0

Comment thread .github/workflows/cont_integration.yml Fixed
Comment thread .github/workflows/cont_integration.yml Fixed
@ValuedMammal ValuedMammal requested a review from tvpeter May 11, 2026 13:21
- Update setup-rbmt action to cargo-rbmt-0.2.1
- Exclude stable toolchain + Cargo-minimal.lock matrix combination
- Bump jsonrpc to 0.20.0
- Bump bitcoind to 0.38.0
- Update stable toolchain to 1.95.0, nightly to floating tag
@luisschwab
Copy link
Copy Markdown
Member

I think it's best to use a vendored GitHub action, since the repo can possibly stop being mirrored to GitHub.

ref: https://github.com/luisschwab/halfin/blob/master/.github/actions/setup-rbmt/action.yml

Create a vendored composite action `.github/actions/setup-rbmt`
that reads the version from the rbmt-version file, falls back to
`master`, installs `cargo-rbmt` from the canonical
rust-bitcoin-maintainer-tools repo, then runs `cargo rbmt toolchains`.

Update `cont_integration.yml` check and test jobs to use
`.github/actions/setup-rbmt` instead of the external pinned SHA.
Copy link
Copy Markdown
Member

@luisschwab luisschwab left a comment

Choose a reason for hiding this comment

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

ACK 5967508

@ValuedMammal ValuedMammal merged commit 7fe50d9 into bitcoindevkit:master May 12, 2026
9 checks passed
@ValuedMammal ValuedMammal deleted the ci/setup-rbmt branch May 12, 2026 01:01
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.

3 participants