We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
F401 delete an import statement used to check whether a module is installed:
try: import slack_sdk del slack_sdk except ImportError as e: raise MissingConditionalImportError('health_checker', 'slack_sdk', None) from e
Code snippet from https://github.com/mosaicml/composer This auto fix just removed the import statement which creates an error when it is run again.
try: del slack_sdk except ImportError as e: raise MissingConditionalImportError('health_checker', 'slack_sdk', None) from e
Obviously, the new code is a bad fix.
The text was updated successfully, but these errors were encountered:
I thought we handled this case (try-catch with ImportError) explicitly. It’s supposed to flag, but not autofix.
Sorry, something went wrong.
Guessing it’s related to the del but I’ll have to test.
charliermarsh
Successfully merging a pull request may close this issue.
Obviously, the new code is a bad fix.
The text was updated successfully, but these errors were encountered: