Fix official build pipeline in dnceng #51496
Conversation
scripts/merge-vs-deps.ps1
Outdated
| git pull origin master-vs-deps | ||
| } | ||
| else { | ||
| git -c http.extraheader="AUTHORIZATION: bearer $accessToken" pull origin master-vs-deps |
There was a problem hiding this comment.
This is the command used by pipeline checkout, so I copied it here and figure it should work.
https://dev.azure.com/dnceng/internal/_build/results?buildId=1010804&view=logs&j=20fcf628-b65c-5865-625a-1cef81cda63b&t=5d97538f-248a-5cb7-616a-d88defa7cde9&l=20
|
@JoeRobich FYI I have made more changes after you signed off |
genlu
changed the title
|
@genlu my only thought is that we may want to rebase these changes and target release/dev16.9 since it will be a LTS release. |
|
@JoeRobich Makes sense. Will do. Who's approval is required? |
No special approvals needed for infrastructure changes. |
Use access token to do master validation
genlu
force-pushed
the
FixAzdoPublish
branch
from
4ff5484
to
ae343a2
Compare
|
@JoeRobich Done |
|
I'd be interested in reviewing once an official build run of this pipeline is available from dnceng as well as from devdiv. |
|
@RikkiGibson I have tested insertion validation change in dnceng, for example: Testing asset publication change would require changes to publishdata, so I will just let the real run test it. Those fixes should have no impact on devdiv pipeline |
ghost
merged commit 
@JoeRobich