Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add exceptions class to bauhaus #91

Open
karishmadaga opened this issue May 18, 2021 · 0 comments · May be fixed by #101
Open

add exceptions class to bauhaus #91

karishmadaga opened this issue May 18, 2021 · 0 comments · May be fixed by #101
Assignees
Labels
testing TDD all day everyday

Comments

@karishmadaga
Copy link
Collaborator

  • improved error handling by adding custom exceptions class for bauhaus (bauhaus/errors.py)
  • removes current use of long strings in classes
  • adds more informative naming of errors instead of the generic ValueError, etc
  • we will be able to catch errors we expect and allow unexpected ones to bubble up during testing and from use
  • errors always follow this pattern: (francois chollet, user experience design for APIs)
    1. Catch user errors early and anticipate common mistakes.
    Do user input validation as soon as possible. Actively keep track of common mistakes that people make, and either solve them by simplifying your API, adding targeted error messages for these mistakes, or having a "solutions to common issues" page in your docs.
    2. Provide detailed feedback messages upon user error.
    A good error message should answer: what happened, in what context? What did the software expect? How can the user fix it? They should be contextual, informative, and actionable. Every error message that transparently provides the user with the solution to their problem means one less support ticket, multiplied by how many times users run into the same issue.
@karishmadaga karishmadaga self-assigned this May 18, 2021
@karishmadaga karishmadaga added the testing TDD all day everyday label May 18, 2021
@karishmadaga karishmadaga linked a pull request Jun 14, 2021 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
testing TDD all day everyday
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant