Open averagehat opened 8 years ago
It needs to be very explicit in what is wrong, that is, it should say exactly what the issue is
Like for the !exists
issue, it sould say something like
Error: Directory X already exists
This is better as a PR against the contracts library.
e.g.
Could become
Do this by raising a
ValueError
rather than returning false. Example:I would just wrap our current boolean-returning functions