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

Remove cryptography from the main install requires #486

Merged
merged 1 commit into from Oct 13, 2017

Conversation

Projects
None yet
2 participants
@jeremycline
Member

jeremycline commented Oct 12, 2017

Cryptography is optional and part of the 'crypto_ng' extra. In the
future it may become a default requirement, but for now only require it
in the extras section.

Signed-off-by: Jeremy Cline jeremy@jcline.org

Remove cryptography from the main install requires
Cryptography is optional and part of the 'crypto_ng' extra. In the
future it may become a default requirement, but for now only require it
in the extras section.

Signed-off-by: Jeremy Cline <jeremy@jcline.org>
@codecov

This comment has been minimized.

codecov bot commented Oct 12, 2017

Codecov Report

Merging #486 into develop will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff            @@
##           develop     #486   +/-   ##
========================================
  Coverage    59.26%   59.26%           
========================================
  Files           29       29           
  Lines         1851     1851           
  Branches       302      302           
========================================
  Hits          1097     1097           
  Misses         667      667           
  Partials        87       87

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 3c55150...3732792. Read the comment docs.

@jeremycline jeremycline merged commit cd7e80a into fedora-infra:develop Oct 13, 2017

3 checks passed

codecov/patch Coverage not affected when comparing 3c55150...3732792
Details
codecov/project 59.26% remains the same compared to 3c55150
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@jeremycline jeremycline deleted the jeremycline:cryptography-in-extras branch Oct 13, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment