Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

support TFS git URLs #45

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

Open
neolynx wants to merge 1 commit into coala:master
base: master
Choose a base branch
Loading
from molior-dbs:feature/tfs-urls
Open

Conversation

@neolynx
Copy link

@neolynx neolynx commented Feb 2, 2020

improving the regex to parse more exotic URLs.

Signed-off-by: André Roth neolynx@gmail.com

@neolynx neolynx requested review from jayvdb and retr0h and removed request for jayvdb February 2, 2020 16:48
improving the regex to parse more exotic URLs.
Signed-off-by: André Roth <neolynx@gmail.com>
Copy link

@retr0h @jayvdb May I know will you merge this PR? We are working on some projects heavily depends on this feature.

neolynx reacted with thumbs up emoji

Copy link

@sobh sobh left a comment

Choose a reason for hiding this comment

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

This patch solves a problem I have with repositories owned by user with an all digits username on GitHub (I know 🤦‍♂️).
E.g.

https://github.com/91861/wayst.git

Thus, the actual username gets wrongfully recognized as the port number. Leaving the owner empty.

Please 🙏 merge.

Copy link
Collaborator

retr0h commented Jul 26, 2023

hi @HaloGithub I was the original author of this project, however, it has been transitioned to the Cola team. While I approve the PR, I probably shouldn't merge it, as I'm not responsible for releasing it. Might be best to reach out to the Cola project owners.

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

Reviewers

@retr0h retr0h Awaiting requested review from retr0h

@jayvdb jayvdb Awaiting requested review from jayvdb

1 more reviewer

@sobh sobh sobh approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /