Skip to content

fix: replace 2 bare except clauses with except Exception#9397

Closed
haosenwang1018 wants to merge 1 commit into
OpenMined:devfrom
haosenwang1018:fix/bare-excepts
Closed

fix: replace 2 bare except clauses with except Exception#9397
haosenwang1018 wants to merge 1 commit into
OpenMined:devfrom
haosenwang1018:fix/bare-excepts

Conversation

@haosenwang1018

Copy link
Copy Markdown

Replace bare except clauses with except Exception.

@rasswanth-s

Copy link
Copy Markdown
Collaborator

Hello 👋

Thanks so much for this PR — I really appreciate the work you put into it.

A quick heads-up: the repo just went through a significant refactor, so the codebase has changed quite a bit from what this PR was built on. As a result, these changes no longer apply cleanly to the new structure, and I'm going to close this PR for now.

if you're interested in contributing to the current version, kindly have a look at the README

Thanks again for contributing — and sorry for the churn!

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