Update to throw exceptions instead of logging errors [np-main]
ixje opened this issue · 1 comments
ixje commented
This is a TODO for a project (see side bar)
Try to remove as much logging errors inside the business logic and raise Exceptions instead. Try to bubble it up to the point the that presentation layer (at this point only a CLI) can present useful and detailed information (if necessary).
- Do not use the generic Exception class but use the more specific exception classes like
ValueError
,TypeError
etc (standard lib exceptions). - Create custom exceptions if needed to give sufficient control to the end-user.
jseagrave21 commented
@ixje are you sure this one is done? I don't see a specific PR that addresses this issue.