-
Notifications
You must be signed in to change notification settings - Fork 199
New issue
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
Make "Must first load config" into exception class #2873
Comments
Thank you for the changes related to addressing this issue! The newly added |
…rror This is in support of #2873 (comment)
Thank you @benclifford , I took a look and thought that looked great for this! |
…rror (#2890) This is in support of #2873 (comment)
Is your feature request related to a problem? Please describe.
This:
should be its own exception class so that code trying to detect uninitialized parsl can detect it based on class, rather than on matching strings.
tagging @yadudoc who had some interest in error structuring.
Additional context
cytomining/CytoTable#93
The text was updated successfully, but these errors were encountered: