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

Recognize GitHub SSH remote URLs that don't start with git@ for hyperlinks #668

Merged
merged 1 commit into from Jul 26, 2021

Conversation

spaarmann
Copy link
Contributor

Closes #667.

SSH remote URLs are usually formatted as git@github.com:user/repo but certain setups can allow using just github.com:user/repo, see issue #667 for an example.

This modifies the regex used to detect a GitHub remote for hyperlink generation to also allow the latter form.

…erlinks

Closes dandavison#667.

SSH remote URLs are usually formatted as `git@github.com:user/repo` but
certain setups can allow using just `github.com:user/repo`, see issue dandavison#667
for an example.
This modifies the regex used to detect a GitHub remote for hyperlink
generation to also allow the latter form.
Copy link
Owner

@dandavison dandavison left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks for dealing with the unit test also.

@dandavison dandavison merged commit c2dc651 into dandavison:master Jul 26, 2021
Kr1ss-XD added a commit to Kr1ss-XD/delta that referenced this pull request Jul 30, 2021
* upstream/master:
  Change example config in README
  Recognize GitHub SSH remote URLs that don't start with `git@` for hyperlinks (dandavison#668)
  Fix the empty result issue in add -p (dandavison#664)
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.

🚀 Show Github commit hyperlinks even if remote URL does not contain git@
2 participants