Skip to content

Add initial support for AzureDevOps repositories.#15

Merged
rhysd merged 5 commits intorhysd:masterfrom
bgianfo:azdevops-support
Nov 3, 2019
Merged

Add initial support for AzureDevOps repositories.#15
rhysd merged 5 commits intorhysd:masterfrom
bgianfo:azdevops-support

Conversation

@bgianfo
Copy link
Copy Markdown
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.

Comment thread src/service.rs Outdated
Comment thread src/service.rs
Comment thread src/service.rs Outdated
@rhysd
Copy link
Copy Markdown
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
Copy Markdown
Contributor Author

bgianfo commented Sep 4, 2019

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

@rhysd
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Contributor Author

bgianfo commented Oct 15, 2019

@rhysd ping

@rhysd
Copy link
Copy Markdown
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
Copy Markdown
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.

2 participants