-
Notifications
You must be signed in to change notification settings - Fork 1.6k
Fixes error when creating cloud patches for Azure repos #3372
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
Conversation
d13
left a comment
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 remove the merge commit and use rebase instead
|
Hey @nzaytsev, while reviewing your PR, I'd suggest the following code changes: 👉 Azure DevOps: need to use
You can also review and apply these suggestions locally on your machine. Learn more about GitKraken Code Suggest
Join your team on GitKraken to speed up PR review. |
08e41ff to
a3f6119
Compare
c8a463b to
6bc7960
Compare
|
Hey @nzaytsev, while reviewing your PR, I'd suggest the following code changes: 👉 Refactored idea of project for remote provider
You can also review and apply these suggestions locally on your machine. Learn more about GitKraken Code Suggest
Join your team on GitKraken to speed up PR review. |
6bc7960 to
76a7fdf
Compare
d13
left a comment
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.
LGTM! Let's merge this to main after the 15.2.1 patch release goes out.
- Uses remote.provider.owner as repoOwnerDomain - Passes project as remote provider to azure repos
366f2d1 to
87232bc
Compare
- Uses remote.provider.owner as repoOwnerDomain - Passes project as remote provider to azure repos
Description
Use remote.provider.owner as repoOwnerDomain
Checklist
Fixes $XXX -orCloses #XXX -prefix to auto-close the issue that your PR addresses