-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
[server] Support 'git@{host}:{user}/{repo}.git' format in context URLs #7951
Conversation
This comment has been minimized.
This comment has been minimized.
/approve |
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.
Not sure, doesn't this raise the false expectation that we use ssh
to checkout the repo?
+1 for the change! WSDYT of trimming to use this magic for GitHub and GitLab only for now? |
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.
I am overruled LGTM
LGTM label has been added. Git tree hash: a4e84bb10a0a63264dc976176caabecf2810a27f
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: jankeromnes, JanKoehnlein Associated issue: #7950 The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
@jankeromnes in addition, we could also rewrite the history/location to make the switch more transparent. |
@JanKoehnlein Interesting point, thanks for raising it. However, I don't see any implications for cloning via HTTPS instead of SSH. To me, if cloning works, that's fantastic, but if it still fails, we're still in the same situation (i.e. we need to support something new). Did you have any specific worries in mind?
@AlexTugarev What are your concerns with Bitbucket and GitLab Self-Managed?
I'd love to somehow make the switch more transparent. 💯 However, what do you mean exactly with "rewrite the history/location"? |
Description
Supports context URLs like
[email protected]:gitpod-io/gitpod.git
by rewriting them tohttps://github.com/gitpod-io/gitpod.git
(without actual SSH authentication)Related Issue(s)
Fixes #7950
How to test
Release Notes
Documentation