-
Notifications
You must be signed in to change notification settings - Fork 258
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
use_codecov_badge()
URL seems incorrect now
#1991
Comments
Ideally we'd just remove the branch piece entirely and show |
Just changed the default branches to |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
i.e. here
usethis/R/coverage.R
Line 48 in da18e11
It generates a link like this:
but if you go to that, it will still show
master
. It does not seem to know how to handle?branch=main
and just ignores it, and then shows what it thinks is the default branch, which ismaster
unless you've changed it in the codecov settings.It seems like this might be a more correct link?
That shows
main
for meThe text was updated successfully, but these errors were encountered: