Skip to content

Add codecov secret to test on PR GitHub CI workflow - expect to see coverage info - #133

Merged
sbillinge merged 5 commits into
diffpy:mainfrom
bobleesj:cov-pr
Nov 6, 2024
Merged

Add codecov secret to test on PR GitHub CI workflow - expect to see coverage info#133
sbillinge merged 5 commits into
diffpy:mainfrom
bobleesj:cov-pr

Conversation

@bobleesj

@bobleesjbobleesj commented Nov 5, 2024

Copy link
Copy Markdown
Contributor

Closes#132

@codecov

codecovBot commented Nov 5, 2024

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 71.07%. Comparing base (726463f) to head (a9da42f).
Report is 1 commits behind head on main.

Additional details and impacted files
@@ Coverage Diff @@## main #133 +/- ##
=======================================
Coverage 71.07% 71.07% =======================================
Files 9 9 Lines 242 242 =======================================
Hits 172 172 Misses 70 70 

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

@bobleesj
bobleesj marked this pull request as ready for review November 5, 2024 23:54
@bobleesj

Copy link
Copy Markdown
ContributorAuthor

@sbillinge Oh yeah, codecov is working in a centralized way now.

In each PR, it uploads codecov info to <branch-name> instead of main!

@bobleesj

Copy link
Copy Markdown
ContributorAuthor

@sbillinge ready for review! - added news

@sbillinge

Copy link
Copy Markdown
Contributor

super!

@bobleesj
bobleesj deleted the cov-pr branch November 6, 2024 12:12
Sign up for freeto 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.

Test implementation of Codecov report and CI in PR

2 participants

@bobleesj@sbillinge