Shivansh-007/piston-cli

Better CLI Error Handling

Shivansh-007 opened this issue · 5 comments

Description

Currently, innocent Piston CLI errors such as "No closing quotation" or "missing required argument" are not being handled by the command error handler as we have no error handler as of now. As a result, the the CLI displays lots of Errors which aren't understandable.

In short we have no custom error error handler, and the default one doesn't emit much useful information. We may do better than this.

Proposed Implementation

Investigate the problem, create logic to handle these types of errors, and open a PR to Piston CLI.

Would you like to implement this yourself?

  • I'd like to implement this feature myself
  • Anyone can implement this feature

Errors to be include in the handled include:

  • NewConnectionError - caused when you are disconnected from the internet.

Screenshot from 2021-04-12 07-49-25

  • AttributeError - When piston-cli/config.yml is present but empty

Screenshot from 2021-04-12 07-49-07

I can look into this.

I can look into this aswell 👍.

I can look into this aswell .

Hey @WANTED10 thanks for showing interest in this issue but it looks like @H4CKY54CK has already being assigned, and this is a rather small issue. You can look into the other issues opened on this repository and get assigned to the one you are interested in.

#50 Closes this.