-
Notifications
You must be signed in to change notification settings - Fork 654
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
Fix issue with pullrequests in Azure DevOps #3181
Fix issue with pullrequests in Azure DevOps #3181
Conversation
@enriqueraso can you update the tests to cover the additional prefixes you added?
|
@arturcic Indeed, I will update it. :) |
@arturcic additional prefixes were added as well a new unit test. |
@enriqueraso, I think if you want to have this included in the next release, better rebase this PR on support/5.x branch. Please include only your commits on top |
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.
Please re-target and rebase on support/5.x
. Besides a few comments, I think this looks good!
0d0d0c4
to
5ab3420
Compare
… and Remotes not well replaced.
100b53d
to
4b18753
Compare
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.
Beautiful! 😍
Thank you @enriqueraso for your contribution! |
Fix issue with PullRequests in Azure DevOps. PullRequest not well detected and Remotes not well replaced.
Description
It detects correctly when a reference name is a PullRequest and also remove Remotes properly on a PullRequest.
Helps on Azure DevOps PullRequest completion. Otherwise it doesn't inherit semver calculated on source branch.
Related Issue
Fix #3180
Motivation and Context
I want to use GitVersion in Azure DevOps with the gitversion.yml described in the bug.
How Has This Been Tested?
I tested it on Azure DevOps adding binaries to the repo and change yml file.
Screenshots:
Checklist: