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

Relax subtle and zeroize requirements #360

Merged
merged 1 commit into from
Aug 28, 2021

Conversation

tarcieri
Copy link
Member

The version was pinned to subtle 2.4.x and zeroize 1.3.x to prevent MSRV breakages, but these crates are comaptible with any version before that as well.

This commit loosens the restrictions to allow compatibility with earlier versions of these crates too. This permits a wider range of combinations.

The version was pinned to `subtle` 2.4.x and `zeroize` 1.3.x to prevent
MSRV breakages, but these crates are comaptible with any version before
that as well.

This commit loosens the restrictions to allow compatibility with earlier
versions of these crates too. This permits a wider range of
combinations.
@codecov-commenter
Copy link

Codecov Report

Merging #360 (e92b33f) into master (5a019ea) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #360   +/-   ##
=======================================
  Coverage   89.47%   89.47%           
=======================================
  Files          39       39           
  Lines        1948     1948           
=======================================
  Hits         1743     1743           
  Misses        205      205           

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 5a019ea...e92b33f. Read the comment docs.

@tarcieri tarcieri merged commit 82d7c9b into master Aug 28, 2021
@tarcieri tarcieri deleted the relax-subtle-and-zeroize-requirements branch August 28, 2021 16:16
This was referenced Aug 28, 2021
This was referenced 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