-
Notifications
You must be signed in to change notification settings - Fork 86
Raise a FileNotFoundError when any config file does not exists #943
Conversation
This PR needs Approvals as follows.
Please choose reviewers and requet reviews! Click to see how to approve each reviewsYou can approve this PR by triggered comments as follows.
See all trigger commentsPlease replace [Target] to review target
|
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.
OA
blueoil/utils/config.py
Outdated
@@ -67,6 +67,11 @@ def _saved_config_file_path(): | |||
if os.path.isfile(filepath): | |||
return filepath | |||
|
|||
raise FileNotFoundError("Config file not found: '{}' nor '{}'".format( |
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.
can you use the same list in the for loop above and here?
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.
Thank you! Fixed.
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.
Readability Approval
/ready |
⏳Merge job is queued... |
Link to any relevant issues or pull requests.
Closes #906 .