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
Add better error handling for empty or invalid org and service env vars #3365
Conversation
…org or service env vars. Should have 100% test coverage.
@jlantz Please look at the linter's suggestions |
@jlantz I believe that this PR has extraneous files in it but I don't want to delete them until I'm confident which version of the files is most up-to-date. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good to me. 👍
Thanks for the contribution! It looks like @prescod is an internal user so signing the CLA is not required. However, we need to confirm this. |
Moved from #3341
Thank you @jlantz !
Adds extra error handling and a new exception, ServiceCannotBeLoaded, to match the exception already there for org load issues. Should have 100% test coverage.
Issues Closed