Fix for "Suppress PSScriptAnalyzer Rule" snippets #5110

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Merged

andyleejordan merged 1 commit into PowerShell:main from TMA-2:main

Dec 3, 2024

Merged

Fix for "Suppress PSScriptAnalyzer Rule" snippets #5110

andyleejordan merged 1 commit into PowerShell:main from TMA-2:main

Dec 3, 2024

Conversation

TMA-2

Copy link

Contributor

@TMA-2 TMA-2 commented Dec 2, 2024 •

edited by JustinGrote

Loading

Fixes #5108. Also my first PR, so hopefully everything checks out!

PR Summary

Corrects a missing close comment in the Function rule snippet
Corrects the tab stop numbering in the Parameter rule snippet as well as duplicate use of $TM_SELECTED_TEXT
Adds a missing comma in The Scope rule snippet
Minor formatting

PR Checklist

Note: Tick the boxes below that apply to this pull request by putting an x between the square brackets.
Please mark anything not applicable to this PR NA.

PR has a meaningful title
Summarized changes
NA PR has tests
This PR is ready to merge and is not work in progress
- If the PR is work in progress, please add the prefix WIP: to the beginning of the title and remove the prefix when the PR is ready

@TMA-2


 Fix for "suppress psscriptanalyzer rule" snippets

4b729a8

Fix for issue PowerShell#5108
- Corrects a missing close comment in the Function rule snippet
- Corrects the tab stop numbering in the Parameter rule snippet as well as duplicate use of `$TM_SELECTED_TEXT`
- Adds a missing comma in The Scope rule snippet
- Minor formatting