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

Use collection expressions to init spans where possible #10114

Merged
merged 1 commit into from
Nov 20, 2023

Conversation

halgab
Copy link
Contributor

@halgab halgab commented Oct 13, 2023

This is the same thing that was done in dotnet/runtime#93125

Proposed changes

  • Use collection expressions to init spans where possible
Microsoft Reviewers: Open in CodeFlow

@ghost ghost assigned halgab Oct 13, 2023
@ghost ghost added the draft draft PR label Oct 13, 2023
@halgab halgab force-pushed the collection-literal branch from 49d7951 to d1a13a2 Compare October 13, 2023 20:07
@halgab halgab marked this pull request as ready for review October 13, 2023 20:39
@halgab halgab requested a review from a team as a code owner October 13, 2023 20:39
@ghost ghost removed the draft draft PR label Oct 14, 2023
@Tanya-Solyanik Tanya-Solyanik added the 📭 waiting-author-feedback The team requires more information from the author label Oct 21, 2023
@ghost ghost removed the 📭 waiting-author-feedback The team requires more information from the author label Oct 21, 2023
@Tanya-Solyanik Tanya-Solyanik added the 📭 waiting-author-feedback The team requires more information from the author label Oct 22, 2023
@ghost ghost removed the 📭 waiting-author-feedback The team requires more information from the author label Oct 22, 2023
@halgab halgab force-pushed the collection-literal branch from 18639ad to 7bbaa42 Compare October 27, 2023 13:25
@Tanya-Solyanik Tanya-Solyanik added the 📭 waiting-author-feedback The team requires more information from the author label Oct 28, 2023
@ghost ghost removed the 📭 waiting-author-feedback The team requires more information from the author label Oct 28, 2023
@halgab halgab force-pushed the collection-literal branch from 1e08310 to 59e3fcf Compare October 29, 2023 15:06
Tanya-Solyanik
Tanya-Solyanik previously approved these changes Oct 30, 2023
@Tanya-Solyanik Tanya-Solyanik added the ready-to-merge PRs that are ready to merge but worth notifying the internal team. label Oct 30, 2023
@Tanya-Solyanik Tanya-Solyanik removed the ready-to-merge PRs that are ready to merge but worth notifying the internal team. label Nov 3, 2023
@Tanya-Solyanik Tanya-Solyanik self-requested a review November 3, 2023 04:06
@Tanya-Solyanik Tanya-Solyanik added the 📭 waiting-author-feedback The team requires more information from the author label Nov 19, 2023
@halgab halgab force-pushed the collection-literal branch from 59e3fcf to 33c534a Compare November 19, 2023 07:51
@ghost ghost removed the 📭 waiting-author-feedback The team requires more information from the author label Nov 19, 2023
@halgab halgab force-pushed the collection-literal branch from 33c534a to d2e830a Compare November 20, 2023 12:15
Tanya-Solyanik added a commit that referenced this pull request Nov 20, 2023
#10335)

Partially reverted https://github.com/dotnet/winforms/pull/9654/files, to move forward PR #10114

After the change, according to the Build.binlog, command line build uses Roslyn binaries that are distributed with arcade - .dotnet\sdk\9.0.100-alpha.1.23511.2\Roslyn\bincore\csc.dll
Microsoft (R) Visual C# Compiler version 4.8.0-3.23477.1 (a0f63522)

Commit associated with this version was merged in September - dotnet/roslyn@a0f63522, after the commit where the inline arrays were introduced, which was in August.
Copy link
Member

@Tanya-Solyanik Tanya-Solyanik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@Tanya-Solyanik Tanya-Solyanik merged commit b673eea into dotnet:main Nov 20, 2023
9 checks passed
@ghost ghost added this to the 9.0 Preview1 milestone Nov 20, 2023
@halgab halgab deleted the collection-literal branch November 20, 2023 18:06
@github-actions github-actions bot locked and limited conversation to collaborators Dec 21, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants