Skip to content

Conversation

@feanil
Copy link
Contributor

@feanil feanil commented Apr 14, 2023

  • build: Update how we upload coverage data.
  • chore: Run make upgrade

@feanil feanil requested a review from a team April 14, 2023 15:38
@codecov
Copy link

codecov bot commented Apr 14, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (b8139aa) 100.00% compared to head (cf0e1bb) 100.00%.

❗ Current head cf0e1bb differs from pull request most recent head 2935ab5. Consider uploading reports for the commit 2935ab5 to get more accurate results

Additional details and impacted files
@@            Coverage Diff            @@
##            master      #274   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           16        16           
  Lines          436       436           
  Branches        47        47           
=========================================
  Hits           436       436           
Flag Coverage Δ
unittests 100.00% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

Feanil Patel added 2 commits April 18, 2023 10:30
The python package is gone, we should be using the latest github action.
@feanil feanil force-pushed the feanil/fix_codecov branch from cf0e1bb to 2935ab5 Compare April 18, 2023 14:40
@feanil feanil changed the title feanil/fix codecov Codecov Fix Apr 18, 2023
@feanil feanil requested a review from timmc-edx April 18, 2023 14:42
@feanil feanil merged commit 2a5d7a5 into master Apr 18, 2023
@feanil feanil deleted the feanil/fix_codecov branch April 18, 2023 14:58
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.

3 participants