Skip to content

fix AvoidUsingEmptyCatchBlock rule violation - #27903

Merged
Dongbo Wang (daxian-dbw) merged 2 commits into
PowerShell:masterfrom
xtqqczze:AvoidUsingEmptyCatchBlock
Sep 29, 2026
Merged

fix AvoidUsingEmptyCatchBlock rule violation#27903
Dongbo Wang (daxian-dbw) merged 2 commits into
PowerShell:masterfrom
xtqqczze:AvoidUsingEmptyCatchBlock

Conversation

@xtqqczze

@xtqqczze xtqqczze commented Aug 29, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Fix AvoidUsingEmptyCatchBlock.md PSScriptAnalyzer rule violation.

Copilot AI lite review requested due to automatic review settings August 29, 2026 17:19
@azure-pipelines

azure-pipelines Bot commented Aug 29, 2026

Copy link
Copy Markdown
Azure Pipelines:
There may be pipelines that require an authorized user to comment /azp run to run.

Copilot AI reviewed Aug 29, 2026

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.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@xtqqczze xtqqczze mentioned this pull request Aug 29, 2026
Open
@xtqqczze
xtqqczze requested a balanced review from Copilot September 6, 2026 14:51
@xtqqczze
xtqqczze force-pushed the AvoidUsingEmptyCatchBlock branch from 5a66360 to be4d704 Compare September 6, 2026 14:51
Copilot AI reviewed Sep 6, 2026
View reviewed changes

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.

🟢 Approval recommended

The change preserves existing fallback behavior while resolving the analyzer violation.

Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 0 new
  • Review effort level: Balanced

@microsoft-github-policy-service microsoft-github-policy-service Bot added the Review - Needed The PR is being reviewed label Sep 25, 2026
@microsoft-github-policy-service

microsoft-github-policy-service Bot commented Sep 25, 2026

Copy link
Copy Markdown
Contributor

This pull request has been automatically marked as Review Needed because it has been there has not been any activity for 7 days.
Maintainer, please provide feedback and/or mark it as Waiting on Author

@daxian-dbw Dongbo Wang (daxian-dbw) added CL-CodeCleanup Indicates that a PR should be marked as a Code Cleanup change in the Change Log and removed Review - Needed The PR is being reviewed labels Sep 28, 2026
@daxian-dbw
Dongbo Wang (daxian-dbw) merged commit 134c00d into PowerShell:master Sep 29, 2026
36 checks passed
@xtqqczze
xtqqczze deleted the AvoidUsingEmptyCatchBlock branch September 29, 2026 19:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot
Copilot review effort
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments
@daxian-dbw Dongbo Wang (daxian-dbw) daxian-dbw approved these changes
@jshigetomi Justin Chung (jshigetomi) Awaiting requested review from jshigetomi jshigetomi is a code owner

Assignees

No one assigned

Labels

CL-CodeCleanup Indicates that a PR should be marked as a Code Cleanup change in the Change Log

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@xtqqczze @daxian-dbw