Skip to content

fix: replace 3 bare except clauses with except Exception#68772

Closed
haosenwang1018 wants to merge 1 commit intosaltstack:masterfrom
haosenwang1018:fix/bare-excepts
Closed

fix: replace 3 bare except clauses with except Exception#68772
haosenwang1018 wants to merge 1 commit intosaltstack:masterfrom
haosenwang1018:fix/bare-excepts

Conversation

@haosenwang1018
Copy link

Replace bare except clauses with except Exception.

@haosenwang1018 haosenwang1018 requested a review from a team as a code owner February 26, 2026 02:03
Copy link
Contributor

@twangboy twangboy left a comment

Choose a reason for hiding this comment

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

Let's figure out what the exceptions actually are that we're tyring to catch. except Exception is just as vague as except.

@haosenwang1018
Copy link
Author

Closing to reduce review backlog; will revisit with smaller patch if needed.

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