Skip to content

Use the AzDO Agent's copy of git#62061

Merged
JoeRobich merged 5 commits intorelease/dev16.11-vs-depsfrom
dev/jorobich/fix-git
Jun 22, 2022
Merged

Use the AzDO Agent's copy of git#62061
JoeRobich merged 5 commits intorelease/dev16.11-vs-depsfrom
dev/jorobich/fix-git

Conversation

@JoeRobich
Copy link
Copy Markdown
Member

Try to use the AzDO agents copy of git to perform the shallow checkout.

@ghost ghost added the Area-Infrastructure label Jun 21, 2022
@JoeRobich JoeRobich marked this pull request as ready for review June 21, 2022 20:21
@JoeRobich JoeRobich requested a review from a team as a code owner June 21, 2022 20:21
Copy link
Copy Markdown
Member

@RikkiGibson RikkiGibson left a comment

Choose a reason for hiding this comment

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

Awesome. Let's just make sure we're ready to address the conflicts when they arise downstream. I wonder if cherry-picking #61965 into this PR then modifying to the current state would help git figure out the right thing to do? Not sure.

@RikkiGibson RikkiGibson self-assigned this Jun 21, 2022

- script: |
@echo on
set PATH=$(Agent.HomeDirectory)\externals\git\cmd;"%PATH%
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

It looks like there's an unmatched " here. I wasn't sure if that was deliberate.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

This is what I get for typing in the web editor.

@RikkiGibson
Copy link
Copy Markdown
Member

Will be happy to drive this but might want to wait until main is reopened.

@JoeRobich
Copy link
Copy Markdown
Member Author

Will be happy to drive this but might want to wait until main is reopened.

Let's flow it as high as it will go.

@JoeRobich JoeRobich merged commit 2846282 into release/dev16.11-vs-deps Jun 22, 2022
@RikkiGibson RikkiGibson deleted the dev/jorobich/fix-git branch June 22, 2022 22:31
@jmarolf jmarolf mentioned this pull request Jun 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants