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
hq worker info <worker-id> #54
Conversation
src/client/worker.rs
Outdated
@@ -45,3 +46,52 @@ pub fn print_worker_info(workers: Vec<WorkerInfo>, gsettings: &GlobalSettings) { | |||
]); | |||
assert!(print_stdout(table).is_ok()); | |||
} | |||
|
|||
pub fn print_worker_details(worker: WorkerDetail, gsettings: &GlobalSettings) { |
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.
Please do not duplicate print_worker_configuration. If you need to modify it, try to refactor by introducing a new function that is shared by both functions. However as it seems it may be just a one function now
src/client/worker.rs
Outdated
@@ -1,8 +1,9 @@ | |||
use cli_table::format::Justify; | |||
use cli_table::{print_stdout, Cell, Color, Style, Table}; | |||
use cli_table::{print_stdout, Cell, Color, Style, Table, Row}; |
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.
Please remove all warnings.
src/bin/hq.rs
Outdated
@@ -123,6 +129,8 @@ enum WorkerCommand { | |||
List(WorkerListOpts), | |||
/// Hwdetect | |||
Hwdetect, | |||
/// Display details about worker | |||
Details(WorkerDetailsOpts), |
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.
I would still prefer "hq worker info " as it is more compatible with the rest of the terminology. However, if there is a good reason why call it "detail", lets discuss it.
See here: #53 (comment) on how to merge changes from |
@User3574 Thank you |
Closes: #22