Skip to content

Conversation

@notmandatory
Copy link
Member

I'm not thrilled about having to pin rustls to meet our MSRV, but we need to make a new release and can't count on rustls/rustls#2239 being accepted by the rustls team.

@notmandatory notmandatory self-assigned this Nov 25, 2024
Copy link
Contributor

@ValuedMammal ValuedMammal left a comment

Choose a reason for hiding this comment

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

utACK b0a1bfb

@luisschwab
Copy link
Member

utACK b0a1bfb

Copy link
Contributor

@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.

ACK b0a1bfb

@notmandatory notmandatory merged commit f00b999 into bitcoindevkit:master Nov 26, 2024
4 checks passed
radiant-smith-lk20 added a commit to radiant-smith-lk20/rust-electrum-client that referenced this pull request Dec 15, 2025
…n for rustls

b0a1bfb0cb2141fb2156af9744965b05fe6976e0 ci: pin msrv dep version for rustls (Steve Myers)

Pull request description:

  I'm not thrilled about having to pin `rustls` to meet our MSRV, but we need to make a new release and can't count on rustls/rustls#2239 being accepted by the `rustls` team.

ACKs for top commit:
  luisschwab:
    utACK b0a1bfb0cb2141fb2156af9744965b05fe6976e0
  ValuedMammal:
    utACK b0a1bfb0cb2141fb2156af9744965b05fe6976e0
  oleonardolima:
    ACK b0a1bfb0cb2141fb2156af9744965b05fe6976e0

Tree-SHA512: 1f37b423f3e10a7e3fa4e2072d5ad38b96341a91f78c1bcffc011dedc08071bd176b556646e40e635c3b558e6b771a4a7baa85b10d14da64d2cd385a69342ee4
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.

4 participants