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

Add initial support for AzureDevOps repositories. #15

Merged
merged 5 commits into from
Nov 3, 2019
Merged

Add initial support for AzureDevOps repositories. #15

merged 5 commits into from
Nov 3, 2019

Conversation

bgianfo
Copy link
Contributor

@bgianfo bgianfo commented Aug 11, 2019

This change implements basic support for repositories
hosted on AzureDevOps (formerly, Visual Studio Team Services).

Because there are many people who still use the old legacy URLs,
which continue to be supported, I've implemented support for both.

This ends up being a matrix of:

I've included tests to validate all of the newly introduced functionality.

src/service.rs Outdated Show resolved Hide resolved
src/service.rs Show resolved Hide resolved
src/service.rs Outdated Show resolved Hide resolved
@rhysd
Copy link
Owner

rhysd commented Aug 12, 2019

@bgianfo Thank you for nice PR! I reviewed the patch and I felt tweaking URL to fit to GitHub-like URLs is hacky. Would you consider to update it?

@bgianfo
Copy link
Contributor Author

bgianfo commented Sep 4, 2019

Sorry for the slow response, I'll get around to addressing your feedback. Thanks!

@rhysd
Copy link
Owner

rhysd commented Sep 5, 2019

No problem. Please take your pace. Please ping me when you finish your work. I'll review changes.

Brian Gianforcaro added 5 commits October 7, 2019 02:34
This change implements basic support for repositories
hosted on AzureDevOps (formerly, Visual Studio Team Services).

Because there are many people who still use the old legacy URLs,
which continue to be supported, I've implemented support for both.

This ends up being a matrix of:
- http://visualstudio.com
- http://dev.azure.com
- ssh://vs-ssh.visualstudio.com
- ssh://ssh.dev.azure.com

I've included tests to validate all of the newly introduced functionality.
@bgianfo
Copy link
Contributor Author

bgianfo commented Oct 8, 2019

@rhysd I think this is ready for another review, thanks!

@bgianfo bgianfo requested a review from rhysd October 11, 2019 01:25
@bgianfo
Copy link
Contributor Author

bgianfo commented Oct 15, 2019

@rhysd ping

@rhysd
Copy link
Owner

rhysd commented Nov 3, 2019

I'm sorry for long delay since I focused on a few projects (and some private projects). Let me check changes.

Copy link
Owner

@rhysd rhysd left a comment

Choose a reason for hiding this comment

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

Looks good! Thank you for the updates.

@rhysd rhysd merged commit 696d6bc into rhysd:master Nov 3, 2019
rhysd added a commit that referenced this pull request Nov 3, 2019
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.

None yet

2 participants