-
Notifications
You must be signed in to change notification settings - Fork 155
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
renv with git-credential-manager #1976
Comments
I opened an issue in the |
It could probably be made more clear, but When installing packages from GitHub using the "github" remote style, e.g. Packages from |
Thanks! True, I may confuse git and Github. And yes, in this specific case I mean a Github repository!
This is where I am confused. I can install remote packages, despite not having GITHUB_PAT nor gitcreds. But I do have my credentials in the credential manager stored via the browser authentication (if I do install gitcreds, they will be shown). That warning message by renv should perhaps not be shown in this scenario? |
It's possible, but my guess is that you might run into GitHub API rate limits since |
I am confused on how ´renv´ deals with packages stored in remote repositories (e.g., Github).
I use Git with SSH keys, but I also have my credentials stored in the
git-credential-manager
. I do not have the env variableGITHUB_PAT
set. Regardless, I can push/commit and use the GH API without problems.However, when I install a remote package in a fresh environment, renv seems to complain that it cannot find any Github credentials and yet packages are installed (without PAT or package
gitcreds
):Is this the intended behavior?
The documentation on how to set up the Github credentials is a bit confusing, I admit. For example, here. you seem to recommend the use of
GITHUB_PAT
, here reads like username + password is recommended, but then you also writeFinally, it is not mentioned that you can authenticate to Github via browser when using git-credential-manager, without the need of creating a PAT first. This makes things much easier.
The text was updated successfully, but these errors were encountered: