-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Configure git to use https instead of ssh. #799
Conversation
Codecov Report
@@ Coverage Diff @@
## master #799 +/- ##
==========================================
- Coverage 72.77% 72.75% -0.02%
==========================================
Files 63 63
Lines 4812 4816 +4
==========================================
+ Hits 3502 3504 +2
- Misses 1051 1052 +1
- Partials 259 260 +1
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🎉
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oops, one thing I realized we missed. We should update the docs for the flag:
Write out a .git-credentials file with the provider user and token to allow
authentication with git over HTTPS. This does write secrets to disk and should
only be enabled in a secure environment.
to
Write out a .git-credentials file with the provider user and token to allow
cloning private modules over HTTPS or SSH. This does write secrets to disk and should
only be enabled in a secure environment.
Ah yep. I've just updated the documentation. So should be all good to go now. |
@lkysow it won't let me mark your requested changes as fixed/completed for some reason. |
@ImperialXT weird. Can you also update |
@lkysow done :) |
🎉 awesome work! I'll try to get a release out soon. :D |
Hello there, Big confused, Do i need to add |
Pretty straightforward. Means that when
write-git-creds
is used it'll work for ssh and https sources without any changes needed. This was discussed in #711