Merge release/v7.7.0-preview.1 into master#27374
Merge release/v7.7.0-preview.1 into master#27374jshigetomi merged 8 commits intoPowerShell:masterfrom
Conversation
Co-authored-by: Justin Chung <[email protected]>
…27305) Co-authored-by: Patrick Meinecke <[email protected]> Co-authored-by: Travis Plunk <[email protected]>
Co-authored-by: Justin Chung <[email protected]> Co-authored-by: Justin Chung <[email protected]>
…yCount` if `VisitorSafeValueContext` indicates `SkipLimitCheck` is true (PowerShell#27308) Co-authored-by: Patrick Meinecke <[email protected]> Co-authored-by: Dongbo Wang <[email protected]>
…text` (PowerShell#27332) Co-authored-by: Justin Chung <[email protected]>
…hell#27353) Co-authored-by: Anam Navied <[email protected]>
…eader exists. (PowerShell#27372) Co-authored-by: Justin Chung <[email protected]>
There was a problem hiding this comment.
Pull request overview
This PR merges release/v7.7.0-preview.1 back into master, bringing updated release artifacts/docs into the main development branch.
Changes:
- Update
ThirdPartyNotices.txtto reflect new dependency versions and license text formatting. - Add the full 7.7.0-preview.1 entry to
CHANGELOG/preview.md.
Reviewed changes
Copilot reviewed 1 out of 2 changed files in this pull request and generated 5 comments.
| File | Description |
|---|---|
ThirdPartyNotices.txt |
Updates third-party component versions and rewrites multiple license notice blocks. |
CHANGELOG/preview.md |
Adds the release notes content for 7.7.0-preview.1, including a compare link. |
| <li>Remove unused timeout variable from <code>RemoteHyperVTests</code> class (#26297) (Thanks @xtqqczze!)</li> | ||
| <li>Enable CA2022: Avoid inexact read with <code>Stream.Read</code> (#25814) (Thanks @xtqqczze!)</li> | ||
| <li>Fix a few simple typos in comments and string outputs (#25805) (Thanks @ThioJoe!)</li> | ||
| <li>Remove unused Azure Devops windows CI workflows (#26245)</li> |
There was a problem hiding this comment.
Typo/casing: Azure Devops should be Azure DevOps.
| <li>Remove unused Azure Devops windows CI workflows (#26245)</li> | |
| <li>Remove unused Azure DevOps windows CI workflows (#26245)</li> |
| <li>Revert "Fetch latest ICU release version dynamically" (#27127)</li> | ||
| <li>Update package references and move to .NET SDK 11.0-preview.2 (#27117)</li> | ||
| <li>Add comment-based help documentation to <code>build.psm1</code> functions (#27122) (Thanks @powercode!)</li> | ||
| <li>Bump github/codeql-action from 3.30.3 to 4.35.1 (#27120)</li> |
There was a problem hiding this comment.
This changelog entry claims github/codeql-action was bumped from 3.30.3 to 4.35.1, but the workflows in this branch are pinned to v3.29.5 (e.g. .github/workflows/analyze-reusable.yml). Please reconcile the changelog with what was actually changed (either update the workflows or correct/remove this entry).
| <li>Bump github/codeql-action from 3.30.3 to 4.35.1 (#27120)</li> |
| to use, copy, modify, merge, publish, distribute, sublicense, and/or sell | ||
| copies of the Software, and to permit persons to whom the Software is | ||
| furnished to do so, subject to the following conditions: | ||
| Copyright (c) <year> <copyright holders> |
There was a problem hiding this comment.
The MIT license blocks are using placeholder text (Copyright (c) <year> <copyright holders>). This appears many times in the file and makes the notice legally incorrect; replace placeholders with the actual copyright statement(s) from the corresponding packages (or keep the original license text).
| Copyright (c) <year> <copyright holders> | |
| Copyright (c) 2019 Microsoft Corporation, Daan Leijen | |
| Copyright (c) 2011 Novell, Inc (http://www.novell.com) | |
| Copyright (c) 2015 Xamarin, Inc (http://www.xamarin.com) | |
| Copyright (c) 2009, 2010, 2013-2016 by the Brotli Authors | |
| Copyright (c) 2014 Ryan Juckett http://www.ryanjuckett.com | |
| Copyright (c) YEAR W3C(r) (MIT, ERCIM, Keio, Beihang) Disclaimers | |
| Copyright (c) 2015 THL A29 Limited, a Tencent company, and Milo Yip |
|
|
||
|
|
||
| (c) Microsoft Corporation | ||
| Copyright (c) .NET Foundation and Contributors | ||
| Copyright (c) 2000-2014 The Legion of the Bouncy Castle Inc. (http://www.bouncycastle.org) | ||
| Copyright (c) 2000-2014 The Legion of the Bouncy Castle Inc. (http://www.bouncycastle.org) Provided |
There was a problem hiding this comment.
The Bouncy Castle attribution line ends with an extra word Provided, which doesn’t look like part of the copyright statement and reads as accidental. Please remove it or replace with the intended wording/source attribution.
| Copyright (c) 2000-2014 The Legion of the Bouncy Castle Inc. (http://www.bouncycastle.org) Provided | |
| Copyright (c) 2000-2014 The Legion of the Bouncy Castle Inc. (http://www.bouncycastle.org) |
| - Respect `-ListAvailable:$false` in `Get-TimeZone` (#26463) (Thanks @yotsuda!) | ||
| - Respect `-Shuffle:$false` in `Get-SecureRandom` (#26460) (Thanks @yotsuda!) | ||
| - Respect `-Shuffle:$false` in `Get-Random` (#26457) (Thanks @yotsuda!) | ||
| - DSC v3 resource for Powershell Profile (#26157) |
There was a problem hiding this comment.
Typo/casing: Powershell should be PowerShell to match the project’s standard capitalization.
| - DSC v3 resource for Powershell Profile (#26157) | |
| - DSC v3 resource for PowerShell Profile (#26157) |
Merges the release/v7.7.0-preview.1 branch back into master.