Skip to content
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

Create convert script for toggling https/git@ #1

Open
ClintLiddick opened this issue Jul 20, 2015 · 4 comments
Open

Create convert script for toggling https/git@ #1

ClintLiddick opened this issue Jul 20, 2015 · 4 comments
Assignees

Comments

@ClintLiddick
Copy link
Member

No description provided.

@psigen
Copy link
Member

psigen commented Jul 20, 2015

What are the reasons not to use just one or the other?

@mkoval
Copy link
Member

mkoval commented Jul 20, 2015

I agree with @psigen. I suggest using https because it is the only option for unauthenticated access.

The only problem I've had with https in the past is the annoyance of having to constantly re-enter my username and password. This is trivially fixed by enabling Git credential caching; i.e.:

git config --global credential.helper cache

@ClintLiddick
Copy link
Member Author

We're using https and only committing that, but I plan to use this on my own pc for development, and I want ssh access, hence the convert script (will just be a sed command most likely). Plus, this way if people use git@ out of habit, it's easy to fix before committing/merging.

@aaronjoh
Copy link
Contributor

I gather from this discussion that the readme notes on "use the included converter script" are aspirational instructions? Or does someone have this script ready to go? If we do include such a script I request that it not be called "convert" because that command is taken. I generally prefer ssh but the credential cache is ok. If we don't include this script, then we need to update the readme.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants