◐ Shell
clean mode source ↗

Fix incorrect `-ReportSummary` Pester test grouping by MatejKafka · Pull Request #2057 · PowerShell/PSScriptAnalyzer

Conversation

@MatejKafka

PR Summary

Fixes grouping of Pester tests where tests for -ReportSummary and a few other parameters were (probably accidentally) nested inside another Describe block. Recommend reviewing the diff locally, GitHub diff rendering is very confused (the commit just removes a level of indentation and moves a single }).

PR Checklist

andyleejordan

Choose a reason for hiding this comment

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

Oh yeah, you're quite right. Thanks!

@andyleejordan andyleejordan changed the title Fix incorrect Invoke-ScriptAnalyzer Pester test grouping Fix incorrect -ReportSummary Pester test grouping

Feb 11, 2025

bergmeister

Choose a reason for hiding this comment

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

Restarted CI check, if that's green, happy with it