-
Notifications
You must be signed in to change notification settings - Fork 77
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 cli for other mephisto commands #822
Conversation
♻️ Removed unused imports & added colors to success/error messages
Codecov Report
@@ Coverage Diff @@
## improve-cli-for-mephisto-wut #822 +/- ##
================================================================
- Coverage 64.66% 64.61% -0.06%
================================================================
Files 107 107
Lines 9281 9281
================================================================
- Hits 6002 5997 -5
- Misses 3279 3284 +5
Continue to review full report at Codecov.
|
…ebookresearch/Mephisto into improve-cli-for-other-mephisto-commands
Replace cli outputs with tables in docs
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM - thanks for these QoL improvements across the codebase!
Just building off of #820
Moves away from the tabulate method in favor of the Table import in
rich
.mephisto requesters
mephisto register
Added red color & provided list of valid providers in case the user forgot
mephisto register <provider_name>
mephisto metrics