Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

elliptic-curve: minimum supported tap for bits is v1.0.1 #1355

Merged
merged 1 commit into from
Oct 2, 2023

Conversation

tarcieri
Copy link
Member

@tarcieri tarcieri commented Oct 2, 2023

...when resolving with -Z minimal-versions

tap is a transitive dependency of bitvec by way of wyz v0.5, which misspecifies the required tap version. This is addressed in newer v0.6 releases of wyz, which bitvec does not yet use.

So, as a workaround, this adds an explicit dep:tap dependency on v1.0.1 for the bits feature.

This was the last remaining minimal-versions incompatibility in this crate, allowing all features to now be tested in the minimal-versions check in CI.

...when resolving with `-Z minimal-versions`

`tap` is a transitive dependency of `bitvec` by way of `wyz` v0.5, which
misspecifies the required `tap` version. This is addressed in newer v0.6
releases of `wyz`, which `bitvec` does not yet use.

So, as a workaround, this adds an explicit `dep:tap` dependency on
v1.0.1 for the `bits` feature.

This was the last remaining `minimal-versions` incompatibility in this
crate, allowing all features to now be tested in the `minimal-versions`
check in CI.
@tarcieri tarcieri merged commit bed2d46 into master Oct 2, 2023
16 checks passed
@tarcieri tarcieri deleted the elliptic-curve/fix-bitvec-minimal-versions branch October 2, 2023 16:14
@tarcieri tarcieri mentioned this pull request Oct 2, 2023
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.

1 participant