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

Suggest the usage of ssh git URLs #28

Closed
wants to merge 1 commit into from
Closed

Suggest the usage of ssh git URLs #28

wants to merge 1 commit into from

Conversation

carusogabriel
Copy link
Contributor

Using the previous ones, it was causing some problems with git push:

fatal: remote error: access denied or repository not exported: /web/qa.git

Using the previous ones, it was causing some problems with `git push`:

```
fatal: remote error: access denied or repository not exported: /web/qa.git
```
@cmb69
Copy link
Member

cmb69 commented Jul 9, 2020

Not sure about changing that generally; I mean, not everybody wants to push, and the way it's described, that can't happen accidentially. Maybe add info about needing the other URLs, if one intends to push?

@carusogabriel
Copy link
Contributor Author

Not sure about changing that generally; I mean, not everybody wants to push, and the way it's described, that can't happen accidentially

@cmb69 Indeed, not everyone wants to push, but if someone does, ssh URLs already provide that "functionality" for us.

In general, ssh over https URLs for Git/GitHub is also safer, that's another reason I'm proposing that :p

Copy link
Member

@cmb69 cmb69 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fair enough. :)

@saundefined
Copy link
Member

@cmb69, I guess it can be closed?

#33

@cmb69
Copy link
Member

cmb69 commented Mar 29, 2021

Ah, right!

@cmb69 cmb69 closed this Mar 29, 2021
@carusogabriel carusogabriel deleted the patch-1 branch March 29, 2021 11:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants