-
Notifications
You must be signed in to change notification settings - Fork 1
Don't hide exceptions #21
Copy link
Copy link
Open
Description
try:
importlib.import_module("smart_open")
except:
print("smart_open not installed, please pip install smart-open")
Hides the actual exception - I had a pyOpenSSL issue with smart_open (just a version issue) and it took me 10 minutes to understand where is it from.
either do raise Exception post print - or even just print it.
but this is something that's a show stopper - so just let it crash.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels