Skip to content

Clarify what -Severity parameter filters#444

Merged
sdwheeler merged 2 commits into
MicrosoftDocs:mainfrom
sdwheeler:sdw-w599297-invoke-pssa-typo
Jul 23, 2026
Merged

Clarify what -Severity parameter filters#444
sdwheeler merged 2 commits into
MicrosoftDocs:mainfrom
sdwheeler:sdw-w599297-invoke-pssa-typo

Conversation

@sdwheeler

@sdwheeler sdwheeler commented Jul 23, 2026

Copy link
Copy Markdown
Collaborator

PR Summary

Clarify what -Severity parameter filters

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.

Copilot AI review requested due to automatic review settings July 23, 2026 17:19
@learn-build-service-prod

Copy link
Copy Markdown

Learn Build status updates of commit 781ebb9:

✅ Validation status: passed

File Status Preview URL Details
reference/ps-modules/PSScriptAnalyzer/Invoke-ScriptAnalyzer.md ✅Succeeded View (ps-modules)

For more details, please refer to the build report.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the documentation for Invoke-ScriptAnalyzer to clarify that the -Severity parameter filters reported rule violations after analysis completes, rather than limiting which rules are executed.

Changes:

  • Clarifies that -Severity filters output only after all rules have run.
  • Explicitly states -Severity does not change which rules are executed, and points to Get-ScriptAnalyzerRule for efficient rule selection.

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

Comment thread reference/ps-modules/PSScriptAnalyzer/Invoke-ScriptAnalyzer.md Outdated
@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.

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@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 d210b00:

✅ Validation status: passed

File Status Preview URL Details
reference/ps-modules/PSScriptAnalyzer/Invoke-ScriptAnalyzer.md ✅Succeeded View (ps-modules)

For more details, please refer to the build report.

@sdwheeler
sdwheeler merged commit 038862d into MicrosoftDocs:main Jul 23, 2026
4 checks passed
@sdwheeler
sdwheeler deleted the sdw-w599297-invoke-pssa-typo branch July 23, 2026 17:24
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