Merge rust-bitcoin/rust-bitcoin#1341: ci: Remove serde version pinning
b05ba16a05
ci: Remove serde version pinning (Tobin C. Harding) Pull request description: The MSRV break in serde is fixed now, remove the serde version pinning. Fix: #1256 ACKs for top commit: Kixunil: ACKb05ba16a05
if CI passes. apoelstra: ACKb05ba16a05
Tree-SHA512: 2046b443500a848cb7b22576473f3b09ece4538dd3d30d7a06b09b28d6fe26f5c4d482f70ec0ec1b79469c221dc9c09e8b05b83debb71fd27f7dc30571cbbcfe
This commit is contained in:
commit
f9146d0391
|
@ -20,9 +20,6 @@ fi
|
||||||
|
|
||||||
# We should not have any duplicate dependencies. This catches mistakes made upgrading dependencies
|
# We should not have any duplicate dependencies. This catches mistakes made upgrading dependencies
|
||||||
# in one crate and not in another (e.g. upgrade bitcoin_hashes in bitcoin but not in secp).
|
# in one crate and not in another (e.g. upgrade bitcoin_hashes in bitcoin but not in secp).
|
||||||
cargo update -p serde --precise 1.0.142
|
|
||||||
cargo update -p serde_test --precise 1.0.142
|
|
||||||
cargo update -p serde_derive --precise 1.0.142
|
|
||||||
duplicate_dependencies=$(cargo tree --target=all --all-features --duplicates | wc -l)
|
duplicate_dependencies=$(cargo tree --target=all --all-features --duplicates | wc -l)
|
||||||
if [ "$duplicate_dependencies" -ne 0 ]; then
|
if [ "$duplicate_dependencies" -ne 0 ]; then
|
||||||
echo "Dependency tree is broken, contains duplicates"
|
echo "Dependency tree is broken, contains duplicates"
|
||||||
|
@ -120,8 +117,5 @@ then
|
||||||
# Adding an empty workspace section excludes this crate from the rust-bitcoin workspace.
|
# Adding an empty workspace section excludes this crate from the rust-bitcoin workspace.
|
||||||
echo '[workspace]\n\n' >> Cargo.toml
|
echo '[workspace]\n\n' >> Cargo.toml
|
||||||
|
|
||||||
cargo update -p serde --precise 1.0.142
|
|
||||||
cargo update -p serde_derive --precise 1.0.142
|
|
||||||
|
|
||||||
cargo test --verbose
|
cargo test --verbose
|
||||||
fi
|
fi
|
||||||
|
|
Loading…
Reference in New Issue