Skip to content
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

Merge 'dotnet/main' into ParamsCollections #72042

Merged

Conversation

AlekseyTs
Copy link
Contributor

No description provided.

dibarbet and others added 30 commits January 31, 2024 18:39
Revert "Merge pull request dotnet#71412 from vibeeshan025/main
…-release/dev17.10

Merge release/dev17.9 to release/dev17.10
…-release/dev17.10

Merge release/dev17.9 to release/dev17.10
Provide overloads to avoid unnecessary boxing
Disable autoBraceSurround on 17.9 P2 tests
…oftJsonVersion

Fix NewtonsoftJsonVersion problem
CyrusNajmabadi and others added 12 commits February 9, 2024 13:03
* Build bootstrap compiler in separate phase

This moves the infrastructure for building a bootstrap compiler into a
separate phase.

The reason for this is to facilitate better performance investigations.
Having the bootstrap compiler build inside our build makes it basically
impossible 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 roslyn

Overall 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.
@AlekseyTs AlekseyTs requested review from a team as code owners February 10, 2024 02:02
@dotnet-issue-labeler dotnet-issue-labeler bot added the untriaged Issues and PRs which have not yet been triaged by a lead label Feb 10, 2024
@AlekseyTs AlekseyTs requested a review from 333fred February 10, 2024 02:03
@AlekseyTs
Copy link
Contributor Author

AlekseyTs commented Feb 10, 2024

@dotnet/roslyn-compiler Need a sign-off on a merge from 'main' into ParamsCollections. This is a fast-forward merge.

@AlekseyTs
Copy link
Contributor Author

@dotnet/roslyn-compiler Need a sign-off on a merge from 'main' into ParamsCollections. This is a fast-forward merge.

@jjonescz
Copy link
Member

This is a fast-forward merge.

I see some tests updated in 25414ee though - are they part of fast-forward merge?

@AlekseyTs
Copy link
Contributor Author

I see some tests updated though - are they part of fast-forward merge?

No, they are not.

@AlekseyTs
Copy link
Contributor Author

One test was adjusted and one new test was added. The change in behavior is expected in the feature branch.

@AlekseyTs AlekseyTs merged commit 1dbe23e into dotnet:features/ParamsCollections Feb 12, 2024
30 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-Compilers Feature - ParamsCollections untriaged Issues and PRs which have not yet been triaged by a lead
Projects
None yet
Development

Successfully merging this pull request may close these issues.