Skip to content

Require CryptoKit without CommonCrypto fallbacks#245

Merged
mattrubin merged 6 commits intodevelopfrom
deployment-targets
Jun 7, 2022
Merged

Require CryptoKit without CommonCrypto fallbacks#245
mattrubin merged 6 commits intodevelopfrom
deployment-targets

Conversation

@mattrubin
Copy link
Copy Markdown
Owner

CryptoKit is only available on iOS 13+ and watchOS 6+

Comment thread Package.swift
Comment thread Package.swift
@codecov
Copy link
Copy Markdown

codecov Bot commented Jun 7, 2022

Codecov Report

Merging #245 (eef2e5d) into develop (48a6ab9) will decrease coverage by 0.20%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop     #245      +/-   ##
===========================================
- Coverage    97.16%   96.96%   -0.21%     
===========================================
  Files            6        6              
  Lines          388      362      -26     
===========================================
- Hits           377      351      -26     
  Misses          11       11              
Impacted Files Coverage Δ
Sources/Crypto.swift 100.00% <ø> (ø)

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 48a6ab9...eef2e5d. Read the comment docs.

@mattrubin
Copy link
Copy Markdown
Owner Author

Codecov's project coverage decrease is due entirely to removed lines of code.

@mattrubin mattrubin merged commit c3d8989 into develop Jun 7, 2022
@mattrubin mattrubin deleted the deployment-targets branch June 7, 2022 18:15
@mattrubin mattrubin added this to the 4.0 milestone Dec 28, 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.

1 participant