Skip to content

Remove subtle pinning#408

Merged
tarcieri merged 1 commit intomasterfrom
unpin-subtle
Mar 19, 2022
Merged

Remove subtle pinning#408
tarcieri merged 1 commit intomasterfrom
unpin-subtle

Conversation

@tarcieri
Copy link
Member

Version pinning to preserve MSRV has proven to be problematic, in that it prevents compatible combinations of crates from working due to artificial restrictions intended to preserve out-of-the-box nameplate MSRV on particular crates.

Since we're bumping MSRV in these crates anyway as part of the cipher upgrade, this commit removes these restrictions, which should allow more compatible combinations of crates to work.

Version pinning to preserve MSRV has proven to be problematic, in that
it prevents compatible combinations of crates from working due to
artificial restrictions intended to preserve out-of-the-box nameplate
MSRV on particular crates.

Since we're bumping MSRV in these crates anyway as part of the `cipher`
upgrade, this commit removes these restrictions, which should allow more
compatible combinations of crates to work.
@codecov-commenter
Copy link

Codecov Report

Merging #408 (555ae1d) into master (6afd561) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #408   +/-   ##
=======================================
  Coverage   87.95%   87.95%           
=======================================
  Files          40       40           
  Lines        1959     1959           
=======================================
  Hits         1723     1723           
  Misses        236      236           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6afd561...555ae1d. Read the comment docs.

@tarcieri tarcieri merged commit e1e35e0 into master Mar 19, 2022
@tarcieri tarcieri deleted the unpin-subtle branch March 19, 2022 16:17
@tarcieri tarcieri mentioned this pull request Jul 30, 2022
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