-
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
Merged
Merged
Changes from all commits
Commits
Show all changes
49 commits
Select commit
Hold shift + click to select a range
f5538cb
Merged PR 9144: [5.0-preview8] Pass RequestAborted to SendFileAsync
Tratcher 609b55f
Merge in 'release/5.0-preview8' changes
dotnet-bot 77435c3
Merge in 'release/5.0-preview8' changes
dotnet-bot 3ad9e57
Merge in 'release/5.0-preview8' changes
dotnet-bot 869d8f5
Merge in 'release/5.0-preview8' changes
dotnet-bot 542031b
Merged PR 9196: Modify WebSockets ValueTaskSource
BrennanConroy 6654c8f
Merge in 'release/5.0-preview8' changes
dotnet-bot ab7a11f
Merge in 'release/5.0-preview8' changes
dotnet-bot f437647
Merge in 'release/5.0-preview8' changes
dotnet-bot c88efce
Merge in 'release/5.0-preview8' changes
dotnet-bot 2bac45c
Merge in 'release/5.0-preview8' changes
dotnet-bot 43b68e3
Merge in 'release/5.0-preview8' changes
dotnet-bot dc3a864
Merge in 'release/5.0-preview8' changes
dotnet-bot 716598e
Merge in 'release/5.0-preview8' changes
dotnet-bot 2827037
Merge in 'release/5.0-preview8' changes
dotnet-bot 670f952
Merge in 'release/5.0-preview8' changes
dotnet-bot 4d23ec0
Merge in 'release/5.0-preview8' changes
dotnet-bot 21a5f39
Merge in 'release/5.0-preview8' changes
dotnet-bot cba2275
Merged PR 9296: [internal/release/5.0-preview8] Update dependencies f…
dotnet-bot 864a292
Merge in 'release/5.0-preview8' changes
dotnet-bot 3e7a106
Merged PR 9371: Revive support for globalization and localization in …
6c665aa
Merge in 'release/5.0-preview8' changes
dotnet-bot f172d20
[internal/release/5.0-preview8] Update dependencies from dnceng/inter…
dotnet-bot 4581307
Merge in 'release/5.0-preview8' changes
dotnet-bot c60c278
Merge in 'release/5.0-preview8' changes
dotnet-bot f22aa71
Merge in 'release/5.0-preview8' changes
dotnet-bot 5dc9931
[internal/release/5.0-preview8] Update dependencies from dnceng/inter…
dotnet-bot fed1953
[internal/release/5.0-preview8] Update dependencies from dnceng/inter…
dotnet-bot d2ccb49
[internal/release/5.0-preview8] Update dependencies from dnceng/inter…
dotnet-bot 06fbd7e
Merge in 'release/5.0-preview8' changes
dotnet-bot 3516b8d
[internal/release/5.0-preview8] Update dependencies from dnceng/inter…
dotnet-bot 6aa8cec
Merge remote-tracking branch 'origin/release/5.0-preview8' into prkri…
pranavkm 7f8e21f
Merged PR 9531: Merge remote-tracking branch 'origin/release/5.0-prev…
a1df872
[internal/release/5.0-preview8] Update dependencies from dnceng/inter…
dotnet-bot 929a89b
[internal/release/5.0-preview8] Update dependencies from dnceng/inter…
dotnet-bot cb4a16a
Merged PR 9643: [internal/release/5.0-preview8] Update dependencies f…
dotnet-bot b8e176e
Merge remote-tracking branch 'origin/release/5.0-preview8' into inter…
pranavkm a4938d0
Merge in 'release/5.0-preview8' changes
dotnet-bot 6205d66
Resolve merge conflicts
wtgodbe 663915b
Resolve last conflict
wtgodbe b32dfa8
Update MonoPlatform.ts
pranavkm f883065
Update InteropTests.csproj
wtgodbe 72a7635
Fix InteropClient testasset
wtgodbe 96b1c23
Merge branch 'wtgodbe/MergePreview8' of https://github.com/dotnet/asp…
wtgodbe e6a163b
Merge branch 'release/5.0' into wtgodbe/MergePreview8
wtgodbe 08309b1
Fix tools.ps1
wtgodbe b02309f
merge
wtgodbe d1ab8ef
Use -nopath
wtgodbe cd0ec1f
Add dotnet-install-scripts files to Helix content
dougbu File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file was deleted.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
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
runtests.*
scripts. Please file a follow-up issue for cleanup after this goes in. Or, just open another PR to remove the redundancy.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 comment
The 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 comment
The reason will be displayed to describe this comment to others. Learn more.
Plz file follow-up issue(s) to
TestRunner
NuGet source additions that should already be in this file (copied from the build and, likely enhanced in the case of internal builds)TestRunner
and theruntests.*
scripts if possibledotnet-install.*
scripts from eng/common/dotnet-install-scripts/ to the Helix payload (placing them in a dotnet-install-scripts/ folder relative to where thetools.*
files landThere 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.
Addressed the third bullet in my commit to your branch @wtgodbe