- Notifications
You must be signed in to change notification settings - Fork4.2k
Merge 'dotnet/main' into ParamsCollections#72042
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Merge 'dotnet/main' into ParamsCollections#72042
Uh oh!
There was an error while loading.Please reload this page.
Conversation
Revert "Merge pull requestdotnet#71412 from vibeeshan025/main
…-release/dev17.10Merge release/dev17.9 to release/dev17.10
…-release/dev17.10Merge release/dev17.9 to release/dev17.10
…o-mainMerge release/dev17.10 to main
…o-mainMerge release/dev17.10 to main
Provide overloads to avoid unnecessary boxing
Disable autoBraceSurround on 17.9 P2 tests
…oslyn into keepProjectCone
…oftJsonVersionFix NewtonsoftJsonVersion problem
* Build bootstrap compiler in separate phaseThis moves the infrastructure for building a bootstrap compiler into aseparate phase.The reason for this is to facilitate better performance investigations.Having the bootstrap compiler build inside our build makes it basicallyimpossible to profile using a freshly built compiler to build our repo.Having separate phases mean that locally I can now- build the bootstrap compiler- start the profiler- rebuild roslynOverall I think it's also just a bit cleaner this way.* yml* yml* cleanup* fixes* more* bootstrap change* yml* more* more
Break out the IsFrozen flag to be an aspect of CompilationTrackerState.
Fix crash with convert to record
AlekseyTs commentedFeb 10, 2024 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
@dotnet/roslyn-compiler Need a sign-off on a merge from 'main' into ParamsCollections. This is a fast-forward merge. |
AlekseyTs commentedFeb 12, 2024
@dotnet/roslyn-compiler Need a sign-off on a merge from 'main' into ParamsCollections. This is a fast-forward merge. |
jjonescz commentedFeb 12, 2024
I see some tests updated in25414ee though - are they part of fast-forward merge? |
AlekseyTs commentedFeb 12, 2024
No, they are not. |
AlekseyTs commentedFeb 12, 2024
One test was adjusted and one new test was added. The change in behavior is expected in the feature branch. |
No description provided.