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

Fix Codecov uploading from GitHub Actions #246

Merged
merged 2 commits into from
Jun 7, 2022
Merged

Fix Codecov uploading from GitHub Actions #246

merged 2 commits into from
Jun 7, 2022

Conversation

mattrubin
Copy link
Owner

The post-v1 Codecov action doesn't automatically find coverage files from Xcode tests.

@codecov
Copy link

codecov bot commented Jun 7, 2022

Codecov Report

Merging #246 (2c9e8f4) into develop (57d2ba9) will increase coverage by 3.09%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop     #246      +/-   ##
===========================================
+ Coverage    94.06%   97.16%   +3.09%     
===========================================
  Files            6        6              
  Lines          354      388      +34     
===========================================
+ Hits           333      377      +44     
+ Misses          21       11      -10     
Impacted Files Coverage Δ
Sources/Generator.swift 100.00% <0.00%> (ø)
Sources/Token+URL.swift 98.41% <0.00%> (+0.15%) ⬆️
Sources/Keychain.swift 92.56% <0.00%> (+0.67%) ⬆️
Sources/Crypto.swift 100.00% <0.00%> (+32.25%) ⬆️

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 57d2ba9...2c9e8f4. Read the comment docs.

@mattrubin mattrubin merged commit 48a6ab9 into develop Jun 7, 2022
@mattrubin mattrubin deleted the codecov branch June 7, 2022 16:07
@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.

None yet

1 participant