Skip to content

Implant cargo-rbmt#203

Merged
luisschwab merged 14 commits into
bitcoindevkit:masterfrom
luisschwab:ci/rbmt
May 3, 2026
Merged

Implant cargo-rbmt#203
luisschwab merged 14 commits into
bitcoindevkit:masterfrom
luisschwab:ci/rbmt

Conversation

@luisschwab
Copy link
Copy Markdown
Member

@luisschwab luisschwab commented May 3, 2026

Closes #178

This PR introduces cargo-rbmt for local and CI checking, linting, testing and lockfile generation.

Changelog

## Added
- Add vendored `setup-rbmt` action (`.github/actions/setup-rbmt/action.yml`)
- Add new CI workflow using `cargo-rbmt` (`.github/workflows/rust.yml`)
- Add `cargo-rbmt`-specific fields on `Cargo.toml`:
  - Define stable and nightly toolchains
  - Define feature sets for testing
  - Allow duplicate dependencies on the tree (`getrandom`, `windows-sys`, `wit-bindgen`)
- Commit `Cargo-recent.lock` and `Cargo-minimal.lock`
- Add Zizmor CI workflow
- Add dependabot on CI

## Removed
- Remove old `.github/workflows/cont_integration.yml` workflow

## Changed
- Update the `justfile` with `cargo-rbmt` recipes
- Fix feature flags on code
- Pin direct and transitive dependencies for MSRV on `Cargo.toml`
- Fix Zizmor warnings and errors on old CI workflows
- Update the `README.md`

@luisschwab luisschwab self-assigned this May 3, 2026
@luisschwab luisschwab added the ci Continuous Integration issues label May 3, 2026
@luisschwab luisschwab requested a review from oleonardolima as a code owner May 3, 2026 03:09
Copy link
Copy Markdown
Collaborator

@oleonardolima oleonardolima left a comment

Choose a reason for hiding this comment

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

utACK 8fc9273

I left a few comments that can be addressed in a follow-up.

Comment thread .clippy.toml
Comment thread Cargo.toml
log = "^0.4"
bitreq = { version = "0.3.4", optional = true }
hex = { version = "0.2.2", package = "hex-conservative" }
bitreq = { version = "0.3.5", optional = true }
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.

nit: now having another look at it it would be better to have both proxy and json-using-serde setup here, they're always being used.

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.

We can look into this in a follow up.

Comment thread justfile
alias a := audit
alias b := build
alias c := check
alias cs := check-sigs
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.

nit: I'd suggest verify-sigs, it could be confused with cargo check one.

Comment thread justfile
Comment thread src/lib.rs
@oleonardolima
Copy link
Copy Markdown
Collaborator

ping @luisschwab :shipit:

@luisschwab luisschwab merged commit e41874d into bitcoindevkit:master May 3, 2026
12 of 13 checks passed
@luisschwab luisschwab deleted the ci/rbmt branch May 3, 2026 17:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci Continuous Integration issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Move CI and local testing to cargo-rbmt

2 participants