Skip to content

Recut - codecov in PR, Codespell, release checklist - #136

Merged
sbillinge merged 2 commits into
diffpy:mainfrom
bobleesj:recut-cov
Nov 8, 2024
Merged

Recut - codecov in PR, Codespell, release checklist#136
sbillinge merged 2 commits into
diffpy:mainfrom
bobleesj:recut-cov

Conversation

@bobleesj

@bobleesjbobleesj commented Nov 8, 2024

Copy link
Copy Markdown
Contributor

Closes#134

@github-actions

Copy link
Copy Markdown

Warning! No news item is found for this PR. If this is a user-facing change/feature/fix,
please add a news item by copying the format from news/TEMPLATE.rst.

@codecov

codecovBot commented Nov 8, 2024

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 91.96%. Comparing base (3352045) to head (b4671ec).
Report is 1 commits behind head on main.

Additional details and impacted files
@@ Coverage Diff @@## main #136 +/- ##
===========================================
+ Coverage 71.07% 91.96% +20.88% 
===========================================
Files 9 7 -2 Lines 242 112 -130 ===========================================
- Hits 172 103 -69 + Misses 70 9 -61 

see 16 files with indirect coverage changes

@bobleesj

Copy link
Copy Markdown
ContributorAuthor

@sbillinge ready for review - py3.13, codecov, pytest, warning resolved.

no additional news added since we've already implemented codecov and spelling check before.

@bobleesj
bobleesj marked this pull request as ready for review November 8, 2024 04:06
@sbillinge
sbillinge merged commit 630ce2c into diffpy:mainNov 8, 2024
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.

Add Codecov Report in PR (new cookiecutting standard)

2 participants

@bobleesj@sbillinge