Skip to content

fix: replace 43 bare excepts with except Exception#477

Merged
SunnyHaze merged 1 commit intoOpenDCAI:mainfrom
haosenwang1018:fix/bare-excepts
Feb 25, 2026
Merged

fix: replace 43 bare excepts with except Exception#477
SunnyHaze merged 1 commit intoOpenDCAI:mainfrom
haosenwang1018:fix/bare-excepts

Conversation

@haosenwang1018
Copy link
Contributor

Replace 43 bare except: with except Exception: across 22 files. Bare except: catches KeyboardInterrupt and SystemExit, masking real errors.

@SunnyHaze
Copy link
Collaborator

Thanks for pointing this out — we sincerely appreciate it.

We'll pay closer attention to this in our future code contributions and make sure it aligns with better coding practices.

Thanks again for helping improve the code quality!

@SunnyHaze SunnyHaze merged commit 22d722c into OpenDCAI:main Feb 25, 2026
9 checks passed
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