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 errors in UI #51

Open
mkretz opened this issue Jun 5, 2022 · 1 comment
Open

Improve errors in UI #51

mkretz opened this issue Jun 5, 2022 · 1 comment
Labels
enhancement New feature or request

Comments

@mkretz
Copy link
Member

mkretz commented Jun 5, 2022

Currently, errors are mostly logged with log.Fatal. We need to integrate them in the UI mechanism properly using an error message.

This task is part of the ongoing decoupling of the checking code from the flow and user interaction code. The basic idea is that every function that currently exists with log.Fatal(...) will return an error instead and the CLI will exit in an orderly manner while logging the fatal error before exit.

High level goals of the refactoring:

  • standard error flow is used
  • checks are more contained (library like) and could be used in a different context (i.e. from a service)
@qasimwarraich
Copy link
Member

Some progress has been made w/ 0a6d36f.

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

No branches or pull requests

2 participants