kaos
kaos copied to clipboard
Remove hardcoded color from cli/kaos_cli/commands/build.py
What is the desired change?
Based on code review last week it would be nice to make terminal color handling in cli/kaos_cli/commands/build.py in the same way, as we have it in other modules (train, serve, etc...):
@build.command(name='deploy',
short_help='{}'.format(
click.style('Build the kaos backend', bold=True, fg='black')))
->
@serve.command(name='deploy',
short_help='Configure and serve an ML model')
How will this change improve kaos?
No point to have command's color hardcoded (maybe apart of kill
command warning).
As we saw on the code review demo - it could be invisible because of a terminal different color scheme.
Any thoughts on the implementation approach?
Refactoring
- The color scheme bug is fixed under the PR #95.
- Also added more description based on newly added feature
![Screen Shot 2020-01-24 at 09 47 32](https://user-images.githubusercontent.com/47716312/73056244-da7b1900-3e8e-11ea-950d-2d2ea4e628b3.png)
![Screen Shot 2020-01-24 at 09 47 46](https://user-images.githubusercontent.com/47716312/73056256-e5ce4480-3e8e-11ea-9e31-997a413fa4c3.png)