Skip to content

Suppress false positive PSScriptAnalyzer warnings in tests and build scripts - #25864

Merged
Ilya (iSazonov) merged 6 commits into
PowerShell:masterfrom
SeeminglyScience:fix-pssa-warnings
Aug 19, 2025
Merged

Ilya (iSazonov) merged 6 commits into
PowerShell:masterfrom
SeeminglyScience:fix-pssa-warnings

Conversation

@SeeminglyScience

Copy link
Copy Markdown
Contributor

PR Summary

PR Context

PR Checklist

# gathered from the GetScript execution.
function Get-TargetResource
{
function Get-TargetResource {

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Style corrections for this file (and the other versions of it) is in it's own commit for easier review FYI

@SeeminglyScience Patrick Meinecke (SeeminglyScience) added the CL-Test Indicates that a PR should be marked as a test change in the Change Log label Aug 14, 2025
@iSazonov

Copy link
Copy Markdown
Collaborator

/azp run PowerShell-CI-linux-packaging, PowerShell-Windows-Packaging-CI

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 2 pipeline(s).

@iSazonov Ilya (iSazonov) self-assigned this Aug 19, 2025
@iSazonov
Ilya (iSazonov) merged commit c5f2446 into PowerShell:master Aug 19, 2025
@microsoft-github-policy-service

microsoft-github-policy-service Bot commented Aug 19, 2025 •

Copy link
Copy Markdown
Contributor

📣 Hey @Patrick Meinecke (@SeeminglyScience), how did we do? We would love to hear your feedback with the link below! 🗣️

🔗 https://aka.ms/PSRepoFeedback

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

Labels

Backport-7.4.x-Done Backport-7.5.x-Done CL-Test Indicates that a PR should be marked as a test change in the Change Log

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants