Skip to content

Update .NET 6 to version 6.0.100-rc.2.21505.57 #16249

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.

Sign up for GitHub

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

Jump to bottom
Merged
daxian-dbw merged 5 commits into from
Oct 15, 2021
Merged

Update .NET 6 to version 6.0.100-rc.2.21505.57 #16249

daxian-dbw merged 5 commits into from
Oct 15, 2021

Conversation

@adityapatwardhan
Copy link
Member

@adityapatwardhan adityapatwardhan commented Oct 14, 2021 •
edited
Loading

PR Summary

Update .NET to the latest version of .NET 6

PR Context

PR Checklist

@ghost ghost assigned daxian-dbw Oct 14, 2021
@daxian-dbw
Copy link
Member

daxian-dbw commented Oct 15, 2021

@adityapatwardhan Let me know if you think this PR is ready to merge.

@iSazonov
Copy link
Collaborator

iSazonov commented Oct 15, 2021

@daxian-dbw Have you taken measurements of the startup scenario yet? Are you planning to? Or are we leaving it as it is?

@adityapatwardhan
Copy link
Member Author

adityapatwardhan commented Oct 15, 2021

@iSazonov I took the measurements for startup with new .NET performance features. No significant benefit and increase in package size. We will investigate further for 7.3.

@adityapatwardhan
Copy link
Member Author

adityapatwardhan commented Oct 15, 2021

@daxian-dbw I want Travis to review before we merge.

@iSazonov
Copy link
Collaborator

iSazonov commented Oct 15, 2021

@iSazonov I took the measurements for startup with new .NET performance features. No significant benefit and increase in package size. We will investigate further for 7.3.

My ask is about PerfView - we could find places for minor perf wins.

TravisEz13
TravisEz13 approved these changes Oct 15, 2021
View reviewed changes
Copy link
Member

@TravisEz13 TravisEz13 left a comment

Choose a reason for hiding this comment

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

fix the one issue and approved

@daxian-dbw daxian-dbw merged commit 45b80a5 into PowerShell:master Oct 15, 2021
@adityapatwardhan adityapatwardhan deleted the UpdateDotNetRC2 branch October 15, 2021 21:30
@iSazonov iSazonov added the CL-BuildPackaging Indicates that a PR should be marked as a build or packaging change in the Change Log label Oct 16, 2021
@daxian-dbw daxian-dbw mentioned this pull request Oct 18, 2021
Closed
22 tasks
daxian-dbw pushed a commit to daxian-dbw/PowerShell that referenced this pull request Oct 21, 2021
@adityapatwardhan @daxian-dbw
Update .NET 6 to version 6.0.100-rc.2.21505.57 (PowerShell#16249)
b6f24cf
@ghost
Copy link

ghost commented Oct 21, 2021

🎉v7.2.0-rc.1 has been released which incorporates this pull request.:tada:

Handy links:

@ghost
Copy link

ghost commented Dec 16, 2021

🎉v7.3.0-preview.1 has been released which incorporates this pull request.:tada:

Handy links:

TrapGodBrim pushed a commit to TrapGodBrim/PowerShell that referenced this pull request Jan 19, 2022
@adityapatwardhan @TrapGodBrim
Update .NET 6 to version 6.0.100-rc.2.21505.57 (PowerShell#16249)
dadaf2a
@TravisEz13 TravisEz13 mentioned this pull request May 24, 2022
Merged
@TravisEz13 TravisEz13 mentioned this pull request Jun 7, 2022
Merged
22 tasks
@TravisEz13 TravisEz13 mentioned this pull request Apr 16, 2024
Closed
@TravisEz13 TravisEz13 mentioned this pull request Jul 9, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@daxian-dbw daxian-dbw daxian-dbw approved these changes

@TravisEz13 TravisEz13 TravisEz13 approved these changes

@anmenaga anmenaga Awaiting requested review from anmenaga

Assignees

@daxian-dbw daxian-dbw

Labels

Backport-7.2.x-Done CL-BuildPackaging Indicates that a PR should be marked as a build or packaging change in the Change Log

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@adityapatwardhan @daxian-dbw @iSazonov @TravisEz13