When connected to a TTY, tell the jq formatter to indent the output, and
enable colorization of the output if the terminal supports it.
Co-authored-by: Mislav Marohnić <mislav@github.com>
Allows multiple issues or PRs to be edited in parallel, and querying for shared fields once to reduce network requests.
Co-authored-by: Sam Coe <samcoe@users.noreply.github.com>
* feat: add web flag to cs list subcommand
web flag only works with repo flag, because,
currently there only param for listing with repo_id
* feat: add web flag to cs crate subcommand
web flag used for creating codespace through web UI instead of terminal.
web flag cannot be used with display-name, idle-timeout,
or retention retention-period
because there's no option for that in the Web UI
* refactor: extract mutual excusive logic to PreRunE
- changed web flag mutual exclusive logic, using cmdutil
- extract that logic to PreRunE clause in createCmd
- move web flag up to make it close to PreRunE clause (for clarity)
- add new param to newCreateCmd fn to facilitate test logic
- apply new newCreateCmd fn to root.go
* fix: clarify flag desc and error message
- remove 'yet' from error messages that can cause misunderstanding
- clarify list web flag can only be used with repo flag
* fix: skip machine check when we use web flag ...
(..and no machine flag provided)
+ add test for this new case
+ adjust related test cases for this new change.
* refactor: move flag check logic to PreRunE
why: err on PreRunE or RunE will also print help if error happened
+ move web, repo, org, user mutual exclusive logic to PreRunE clause
+ move repo, org, user mutual exclusive logic to PreRunE
+ move limit check flag to PreRunE
+ modify newListCmd fn to facilitate test logic
+ apply new newListCmd to root.go
+ add test cases to check PreRunE clause
- remove mutual exclusive test cases from Test_AppList
* refactor: remove the opts equality checks
* fix: mutually exclusive misfires
because of wrong logic
+ refine test case too
* cleanup:removing useWeb check in fn getMachineName
because it's no longer needed
+ remove redundant test-case
* refactor: remove redundant ifs
* refactor: clarify test name
* re-clarify web flag desc in list.go
* refactor: break long lines, use more idiomatic err
* add test case for nonexistent/wrong machine
