Skip to content

Conversation

@tcharding
Copy link
Member

@tcharding tcharding commented Jul 12, 2023

Pin serde_json for 1.41 and 1.47 CI runs. Remove the pinning docs from the README and instead point devs to the CI script. (The docs are stale currently and are an unnecessary maintenance burden.)

As usual crates in the Rust ecosystem don't give a f**k about semver
rules.

Pin `serde_json` for 1.41 and 1.47 CI runs. Remove the pinning docs from
the README and instead point devs to the CI script. (The docs are stale
currently and are an unnecessary maintenance burden.)
@tcharding tcharding force-pushed the 07-12-pin-serde_json branch from 9daec76 to 6f54ed9 Compare July 12, 2023 05:43
Copy link
Member

@apoelstra apoelstra left a comment

Choose a reason for hiding this comment

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

ACK 6f54ed9

@apoelstra apoelstra merged commit 1f7f575 into rust-bitcoin:master Jul 12, 2023
@tcharding tcharding deleted the 07-12-pin-serde_json branch July 13, 2023 04:09
heap-coder added a commit to heap-coder/rust-miniscript that referenced this pull request Sep 27, 2025
6f54ed921be62b93ecb4c44fadeff60cac1d85ca Pin serde_json (Tobin C. Harding)

Pull request description:

  Pin `serde_json` for 1.41 and 1.47 CI runs. Remove the pinning docs from the README and instead point devs to the CI script. (The docs are stale currently and are an unnecessary maintenance burden.)

ACKs for top commit:
  apoelstra:
    ACK 6f54ed921be62b93ecb4c44fadeff60cac1d85ca

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

2 participants