-
Notifications
You must be signed in to change notification settings - Fork 129
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
Codecov badge should reflect test coverage of currently used git branch [FR] #1086
Labels
Comments
ilyaZar
added a commit
to ilyaZar/golem
that referenced
this issue
Aug 9, 2023
- to clearly show what the testing coverage is on each branch via the badge - `README.Rmd` is altered to programatically retrieve the current git branch which automatically uses the correct coverage results from https://about.codecov.io/ Refs: ThinkR-open#1086
ColinFay
pushed a commit
that referenced
this issue
Feb 28, 2024
- to clearly show what the testing coverage is on each branch via the badge - `README.Rmd` is altered to programatically retrieve the current git branch which automatically uses the correct coverage results from https://about.codecov.io/ Refs: #1086
ColinFay
pushed a commit
that referenced
this issue
Feb 28, 2024
- to clearly show what the testing coverage is on each branch via the badge - `README.Rmd` is altered to programatically retrieve the current git branch which automatically uses the correct coverage results from https://about.codecov.io/ Refs: #1086
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Is your feature request related to a problem?
The coverage status badge always reports code coverage for the
master
branch (as the links are hard-coded) even when switching to a different branch such asdev
.This may be misleading and makes it difficult to (quickly) see the testing coverage when implementing new features to e.g.
dev
-branch.Describe the solution you'd like
Change the badge links (webpage-link + svg-image-link) inside
README.Rmd
to the currently used branch (for all branches of{golem}
).Describe alternatives you've considered
README
page and see the output ofcovr::package_coverage()
but it takes more time; a badge at the top would make this quickerAdditional context
Can be fixed via #1087
The text was updated successfully, but these errors were encountered: