Skip to content

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

Merged
daxian-dbw merged 5 commits intoPowerShell:masterfrom
adityapatwardhan:UpdateDotNetRC2
Oct 15, 2021
Merged

Update .NET 6 to version 6.0.100-rc.2.21505.57#16249
daxian-dbw merged 5 commits intoPowerShell:masterfrom
adityapatwardhan:UpdateDotNetRC2

Conversation

@adityapatwardhan
Copy link
Copy Markdown
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
Copy Markdown
Member

daxian-dbw commented Oct 15, 2021

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

@iSazonov
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Member Author

adityapatwardhan commented Oct 15, 2021

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

@iSazonov
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown

ghost commented Oct 21, 2021

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

Handy links:

@ghost
Copy link
Copy Markdown

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