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

abstract cli ui utilities, move away from logger.info for display #5347

Closed
skshetry opened this issue Jan 27, 2021 · 1 comment
Closed

abstract cli ui utilities, move away from logger.info for display #5347

skshetry opened this issue Jan 27, 2021 · 1 comment
Assignees
Labels
research ui user interface / interaction

Comments

@skshetry
Copy link
Member

It's hard to describe, but we are trying to move away from logging for UI related prints.
To do so, we have to start consolidating the utilities in one place, decide default colours for the output, reuse certain components (table, etc.).

Then slowly, start replacing the usages of logger internally for printing.

@skshetry skshetry self-assigned this Jan 27, 2021
@skshetry skshetry changed the title abstract cli ui utilities abstract cli ui utilities, move away from logger.info for display Jan 27, 2021
@dberenbaum dberenbaum added the ui user interface / interaction label May 24, 2021
@skshetry
Copy link
Member Author

Closing, as we have moved away from logger.info in the CLI, and created some abstractions. Other stuff will be added as I go through the commands based on necessity. :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
research ui user interface / interaction
Projects
None yet
Development

No branches or pull requests

2 participants