Skip to content

[Freshness] PSScriptAnalyzer Article Updates - Batch 4#405

Open
Xelu86 wants to merge 2 commits into
MicrosoftDocs:mainfrom
Xelu86:AvoidBatch4
Open

[Freshness] PSScriptAnalyzer Article Updates - Batch 4#405
Xelu86 wants to merge 2 commits into
MicrosoftDocs:mainfrom
Xelu86:AvoidBatch4

Conversation

@Xelu86
Copy link
Copy Markdown
Contributor

@Xelu86 Xelu86 commented Jun 1, 2026

PR Summary

These updates target the following PSScriptAnalyzer articles:

  • AvoidUsingCmdletAliases
  • AvoidUsingComputerNameHardcoded
  • AvoidUsingConvertToSecureStringWithPlainText
  • AvoidUsingDeprecatedManifestFields
  • AvoidUsingDoubleQuotesForConstantString

Fixes: AB#581964
Fixes: AB#581965
Fixes: AB#581967
Fixes: AB#581968
Fixes: AB#583951

PR Checklist

  • Descriptive Title: This PR's title is a synopsis of the changes it proposes.
  • Summary: This PR's summary describes the scope and intent of the change.
  • Contributor's Guide: I have read the contributors guide.
  • Style: This PR adheres to the style guide.

@learn-build-service-prod
Copy link
Copy Markdown

PoliCheck Scan Report

The 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 found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

@learn-build-service-prod
Copy link
Copy Markdown

Learn Build status updates of commit 5fef2ee:

💡 Validation status: suggestions

File Status Preview URL Details
reference/docs-conceptual/PSScriptAnalyzer/Rules/AvoidUsingConvertToSecureStringWithPlainText.md 💡Suggestion View (ps-modules) Details
reference/docs-conceptual/PSScriptAnalyzer/Rules/AvoidUsingCmdletAliases.md ✅Succeeded View (ps-modules)
reference/docs-conceptual/PSScriptAnalyzer/Rules/AvoidUsingComputerNameHardcoded.md ✅Succeeded View (ps-modules)
reference/docs-conceptual/PSScriptAnalyzer/Rules/AvoidUsingDeprecatedManifestFields.md ✅Succeeded View (ps-modules)
reference/docs-conceptual/PSScriptAnalyzer/Rules/AvoidUsingDoubleQuotesForConstantString.md ✅Succeeded View (ps-modules)

reference/docs-conceptual/PSScriptAnalyzer/Rules/AvoidUsingConvertToSecureStringWithPlainText.md

  • Line 15, Column 1: [Suggestion: preserve-view-not-set - See documentation] You've pinned this link to a specific version of content with the view parameter. It's recommended not to pin a version unless that version is A) not the default view and B) the context is about that version specifically. To proceed with pinning a version add the &preserve-view=true to the URL. Otherwise, remove the view parameter. URL: /dotnet/api/system.security.securestring?view=net-10.0

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

@learn-build-service-prod
Copy link
Copy Markdown

PoliCheck Scan Report

The 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 found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

@learn-build-service-prod
Copy link
Copy Markdown

@Xelu86 Xelu86 marked this pull request as ready for review June 1, 2026 18:54
Copilot AI review requested due to automatic review settings June 1, 2026 18:54
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Updates several PSScriptAnalyzer rule reference docs to refresh wording, replace "Wrong"/"Correct" headings with "Noncompliant"/"Compliant", and bump ms.date values.

Changes:

  • Reworded descriptions for clarity across multiple rule docs.
  • Renamed example section headings from Wrong/Correct to Noncompliant/Compliant.
  • Updated ms.date to 06/01/2026 and added a link reference for SecureString.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

Show a summary per file
File Description
reference/docs-conceptual/PSScriptAnalyzer/Rules/AvoidUsingDoubleQuotesForConstantString.md Clarified description and renamed example headings.
reference/docs-conceptual/PSScriptAnalyzer/Rules/AvoidUsingDeprecatedManifestFields.md Rewrote description, removed "How" section, renamed headings.
reference/docs-conceptual/PSScriptAnalyzer/Rules/AvoidUsingConvertToSecureStringWithPlainText.md Expanded description, added SecureString link reference.
reference/docs-conceptual/PSScriptAnalyzer/Rules/AvoidUsingComputerNameHardcoded.md Reworded description, updated example casing, renamed headings.
reference/docs-conceptual/PSScriptAnalyzer/Rules/AvoidUsingCmdletAliases.md Reworded description, restructured Configure rule section.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants