You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
A PacmanConfigurationException class sounds nice. 👍 But I'm not so sure about making it checked. It sounds like it's an Exception from which you cannot recover, in which case you're just littering your code.
Unchecked exceptions represent conditions that, generally speaking, reflect errors in your program's logic and cannot be reasonably recovered from at run time
Currently, incorrect configuration can throw IOException and IllegalArgumentException, which in some cases are wrapped into RuntimeException.
My proposal is to introduce one top level checked PacmanConfigurationException, which can replace/wrap the existing exceptions.
/cc @rsouzacoelho
The text was updated successfully, but these errors were encountered: