bitcoindevkit / bdk / 12895009404
79%

DEFAULT BRANCH: master
Ran
Jobs 1
Files 51
Run time 1min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 85.956%. Remained the same
12895009404

push

github

notmandatory 
Merge bitcoindevkit/bdk#1803: ci: add secondary MSRV job of 1.75.0

c8b55da39 chore: fix punctuation and use `rustup override set` instead (Leonardo L.)
78bbfbe72 ci: add `pin-msrv.sh` (valued mammal)
f7c9bc516 docs: update `README.md` with MSRV policy (valued mammal)
12eabd1ec chore(cleanup): remove rustls from pinned deps (Leonardo Lima)
e3faeb63a ci: add secondary MSRV job of 1.75.0 (Leonardo Lima)

Pull request description:

  partially fixes #1750

  <!-- You can erase any parts of this template not applicable to your Pull Request. -->

  ### Description

  <!-- Describe the purpose of this PR, what's being adding and/or fixed -->

  It adds a secondary MSRV version of `1.75.0` for the `bdk_electrum` crate, excluding it from the `1.63.0` MSRV dependency pinning and CI step.

  As it's the new CI will run without `bdk_electrum` for `1.63.0` MSRV step, will run another step for `1.75.0` MSRV and the latest stable.

  Currently,, the only affected crate is `electrum-client`, therefore `bdk-electrum`, as it relies directly on `rustls` which migrated to `1.71.0` MSRV. The `esplora-client` relies on `rustls` as a dependency of `minreq` or `reqwest`, but those crates didn't upgraded it's `rustls` versions yet.

  In a further PR it should also bump the `electrum-client` crate to it's latest version, which relies on `1.75.0` MSRV, see https://github.com/bitcoindevkit/rust-electrum-client/pull/159.

  ### Notes to the reviewers

  <!-- In this section you can include notes directed to the reviewers, like explaining why some parts
  of the PR were done in a specific way -->

  It's open for discussion if this approach is the expected one, or another one would be better.

  ### Changelog notice

  <!-- Notice the release manager should include in the release tag message changelog -->
  <!-- See https://ke... (continued)

11966 of 13921 relevant lines covered (85.96%)

20521.08 hits per line

Jobs
ID Job ID Ran Files Coverage
1 12895009404.1 51
85.96
 GitHub Action Run
Source Files on build 12895009404