-
Notifications
You must be signed in to change notification settings - Fork 10.4k
Merge preview8 branch to public #25280
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
Changes from 47 commits
f5538cb
609b55f
77435c3
3ad9e57
869d8f5
542031b
6654c8f
ab7a11f
f437647
c88efce
2bac45c
43b68e3
dc3a864
716598e
2827037
670f952
4d23ec0
21a5f39
cba2275
864a292
3e7a106
6c665aa
f172d20
4581307
c60c278
f22aa71
5dc9931
fed1953
d2ccb49
06fbd7e
3516b8d
6aa8cec
7f8e21f
a1df872
929a89b
cb4a16a
b8e176e
a4938d0
6205d66
663915b
b32dfa8
f883065
72a7635
96b1c23
e6a163b
08309b1
b02309f
d1ab8ef
cd0ec1f
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
This file was deleted.
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -24,6 +24,15 @@ | |
<ItemGroup Condition="$(BuildHelixPayload)"> | ||
<Content Include="@(HelixContent)" /> | ||
<Content Include="$(RepoRoot)eng\scripts\Download.ps1" /> | ||
<Content Include="$(RepoRoot)NuGet.config" /> | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I realized after doing this that we probably don't need to add as many sources in eng/helix/content/RunTests/TestRunner.cs anymore. That file also adds a bunch of environment variables that are set in the I'm keeping this separate because it's not really part of the merge. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. /cc @JunTaoLuo There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Plz file follow-up issue(s) to
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Addressed the third bullet in my commit to your branch @wtgodbe |
||
<Content Include="$(RepoRoot)global.json" /> | ||
<ContentWithTargetPath Include=" | ||
$(RepoRoot)eng\common\pipeline-logging-functions.*; | ||
$(RepoRoot)eng\common\tools.*"> | ||
<TargetPath>eng\common\%(Filename)%(Extension)</TargetPath> | ||
<CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory> | ||
<CopyToPublishDirectory>PreserveNewest</CopyToPublishDirectory> | ||
</ContentWithTargetPath> | ||
</ItemGroup> | ||
|
||
<!-- | ||
|
Uh oh!
There was an error while loading. Please reload this page.