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

Improve error handling in the Kraken cli for a better user experience #49

Merged
merged 5 commits into from
Jun 22, 2023

Conversation

NiklasRosenstein
Copy link
Contributor

@NiklasRosenstein NiklasRosenstein commented Jun 22, 2023

Removed use of pretty_errors again as the output was confusing people more than the standard Python traceback.

If a task is specified on the CLI that doesn't exist:

Screenshot 2023-06-22 at 12 35 49

If an error occurs within the build script:

Screenshot 2023-06-22 at 12 39 54

If an error occurs in the CLI directly:

Screenshot 2023-06-22 at 12 40 22

NiklasRosenstein and others added 5 commits June 22, 2023 12:21
…ce has shown that it does not provide the added value that we hoped for as it still just outputs a Python traceback, but now in a different format than people are used to.
…riable to allow the Kraken-Core CLI to detect if it is run through `krakenw`.
@NiklasRosenstein NiklasRosenstein merged commit 729960b into develop Jun 22, 2023
10 checks passed
@NiklasRosenstein NiklasRosenstein deleted the nr/catch-address-resolution-error branch June 22, 2023 11:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant