Code review comment for ~ycheng-twn/checkbox-ng:0001_checkbox_cli_show

Revision history for this message
Maciej Kisielewski (kissiel) wrote :

The branch uses unsupported syntax - Xenial runs python 3.5 which doesn't support f-strings.

Also worth noting is that a similar information is printed with:

checkbox-cli list -a all_jobs (also prints info about templates)

and

checkbox-cli list -a job (prints info only about concrete jobs)

Also, check the comments below.
Functions defined in this MR are easy to be tested with UTs.

« Back to merge proposal