Minor edits to pre-77 content#12938
Conversation
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans. ✅ No issues foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
|
Learn Build status updates of commit 3436024: ✅ Validation status: passed
For more details, please refer to the build report. |
There was a problem hiding this comment.
Pull request overview
Note
Copilot was unable to run its full agentic suite in this review.
Updates pre-7.7 PowerShell documentation pages with small consistency edits, including refreshed metadata dates and minor content/link adjustments.
Changes:
- Update
ms.datefront-matter to reflect recent edits. - Normalize/rewrap some prose and list formatting in
ConvertTo-Jsondocs across versions. - Adjust links (add 7.6 changelog reference; change
about_CommonParameterstarget in some versions) and removeJoin-Path -Extensiondocumentation from the 7.6 page.
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| reference/docs-conceptual/whats-new/What-s-New-in-PowerShell-76.md | Updates ms.date and fixes the changelog link reference to point to the 7.6 changelog. |
| reference/7.6/Microsoft.PowerShell.Utility/ConvertTo-Json.md | Reflows text, updates version wording for BigInteger note, adjusts list formatting, and changes about_CommonParameters link. |
| reference/7.6/Microsoft.PowerShell.Management/Join-Path.md | Removes -Extension from syntax, examples, and parameter documentation. |
| reference/7.5/Microsoft.PowerShell.Utility/ConvertTo-Json.md | Reflows text and updates version wording for BigInteger note. |
| reference/7.4/Microsoft.PowerShell.Utility/ConvertTo-Json.md | Reflows text, adds a BigInteger note, and changes about_CommonParameters link. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| As of PowerShell 7.1, `ConvertTo-Json` emits a warning if the depth of the input object exceeds the | ||
| depth specified for the command. This prevents unwanted data loss when converting objects. | ||
|
|
||
| As of PowerShell 7.5, `ConvertTo-Json` can serialize **BigInteger** values as raw JSON numbers. |
There was a problem hiding this comment.
This is the PowerShell 7.4 reference page, but it now documents a behavior introduced in PowerShell 7.5. That can confuse readers who expect the page to describe 7.4 behavior. Consider removing this sentence from the 7.4 page, or rewriting it to clearly indicate it’s not available in 7.4 and pointing readers to the 7.5+ documentation instead.
| As of PowerShell 7.5, `ConvertTo-Json` can serialize **BigInteger** values as raw JSON numbers. | |
| In PowerShell 7.4, `ConvertTo-Json` doesn't serialize **BigInteger** values as raw JSON numbers. | |
| This behavior was introduced in PowerShell 7.5. |
|
|
||
| This example shows the output from the `ConvertTo-Json` cmdlet with and without the **AsArray** | ||
| `[switch]` parameter. You can see the second portion of the output is wrapped in array brackets. | ||
| This example shows the output from `ConvertTo-Json` cmdlet with and without the **AsArray** switch |
There was a problem hiding this comment.
Grammar regression: 'output from ConvertTo-Json cmdlet' should be 'output from the ConvertTo-Json cmdlet'. Adjusting this keeps the sentence correct and consistent with typical cmdlet documentation phrasing.
| This example shows the output from `ConvertTo-Json` cmdlet with and without the **AsArray** switch | |
| This example shows the output from the `ConvertTo-Json` cmdlet with and without the **AsArray** switch |
| -InformationAction, -InformationVariable, -OutVariable, -OutBuffer, -PipelineVariable, -Verbose, | ||
| -WarningAction, and -WarningVariable. For more information, see | ||
| [about_CommonParameters](../Microsoft.PowerShell.Core/About/about_CommonParameters.md). | ||
| [about_CommonParameters](https://go.microsoft.com/fwlink/?LinkID=113216). |
There was a problem hiding this comment.
This changes about_CommonParameters to a fwlink in 7.6 (and similarly in 7.4), but 7.5 appears to keep the repo-relative link. For consistency (and to keep navigation/versioning behavior uniform across the versioned reference set), consider standardizing on one approach across 7.4/7.5/7.6—either keep the relative link everywhere or switch all versions to the fwlink.
| [about_CommonParameters](https://go.microsoft.com/fwlink/?LinkID=113216). | |
| [about_CommonParameters](../Microsoft.PowerShell.Core/About/about_CommonParameters.md). |
PR Summary
Minor edits to pre-77 content
PR Checklist