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

Setup proper Error handling #92

Closed
gromdimon opened this issue Apr 30, 2024 · 0 comments · Fixed by #94
Closed

Setup proper Error handling #92

gromdimon opened this issue Apr 30, 2024 · 0 comments · Fixed by #94
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@gromdimon
Copy link
Collaborator

Is your feature request related to a problem? Please describe.
Currently the script has multiple places, where various errors can occure. However, we need to have clearly specified error handling behaviour.

Describe the solution you'd like
Go thorugh all methods and a) find places where built-in errors can occure b) find places where custom errors can occure.
Then we need to implement proper error handling with logging just on the highest stage (in CLI app)

Describe alternatives you've considered
N/A

Additional context
Probably we need to create more custom errors and re-raise built-in ones in the building blocks.

@gromdimon gromdimon added the enhancement New feature or request label Apr 30, 2024
@gromdimon gromdimon added this to the 0.1.0 milestone Apr 30, 2024
@gromdimon gromdimon self-assigned this Apr 30, 2024
@gromdimon gromdimon changed the title Stup proper Error handling Setup proper Error handling Apr 30, 2024
@gromdimon gromdimon linked a pull request May 2, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant