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

ci: use make coverage #28

Merged
merged 6 commits into from
Mar 26, 2024
Merged

ci: use make coverage #28

merged 6 commits into from
Mar 26, 2024

Conversation

fmorency
Copy link
Collaborator

@fmorency fmorency commented Mar 25, 2024

Use make coverage and upload filtered results to codecov.

Image caching and optimization can be part of future work.

Copy link

codecov bot commented Mar 26, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

❗ No coverage uploaded for pull request base (main@ef4fc76). Click here to learn what that means.

❗ Current head 81734fb differs from pull request most recent head 67a77ad. Consider uploading reports for the commit 67a77ad to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #28   +/-   ##
=======================================
  Coverage        ?   42.80%           
=======================================
  Files           ?       34           
  Lines           ?     2196           
  Branches        ?        0           
=======================================
  Hits            ?      940           
  Misses          ?     1212           
  Partials        ?       44           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@fmorency fmorency self-assigned this Mar 26, 2024
@fmorency fmorency added enhancement New feature or request ci labels Mar 26, 2024
@fmorency fmorency marked this pull request as ready for review March 26, 2024 13:17
@Reecepbcups Reecepbcups merged commit bb2ed3d into liftedinit:main Mar 26, 2024
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants