NotificationsYou must be signed in to change notification settings
Fork928
Star10.1k

fix: Group subcommands for cognitive ease#1351

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Jump to bottom

Merged

kylecarbs merged 1 commit intomainfromseparatecmd

May 9, 2022

Merged

fix: Group subcommands for cognitive ease#1351

kylecarbs merged 1 commit intomainfromseparatecmd

May 9, 2022

Conversation

Copy link

Member

kylecarbs commentedMay 9, 2022•
edited
Loading

Fixes#1324.

I removed the ASCII logo too. It was cool, but not worth the prime real estate.

kylecarbs requested a review fromcoadler

May 9, 2022 20:43

kylecarbs self-assigned this

May 9, 2022

kylecarbs force-pushed theseparatecmd branch from15a2af5 to4f27c4bCompare

May 9, 2022 20:44

kylecarbs requested a review fromammario

May 9, 2022 20:57

Copy link

codecovbot commentedMay 9, 2022•
edited
Loading

Codecov Report

Merging#1351 (8ea93b2) intomain (f5693df) willincrease coverage by0.32%.
The diff coverage is100.00%.

@@            Coverage Diff             @@##             main    #1351      +/-   ##==========================================+ Coverage   66.58%   66.90%   +0.32%==========================================  Files         284      280       -4       Lines       18593    18466     -127       Branches      235      235              ==========================================- Hits        12380    12355      -25+ Misses       4950     4845     -105- Partials     1263     1266       +3

Flag	Coverage Δ
unittest-go-macos-latest	`54.06% <100.00%> (+0.33%)`	⬆️
unittest-go-postgres-	`65.64% <100.00%> (+0.49%)`	⬆️
unittest-go-ubuntu-latest	`56.48% <100.00%> (+0.21%)`	⬆️
unittest-go-windows-2022	`52.42% <100.00%> (+0.35%)`	⬆️
unittest-js	`73.67% <ø> (ø)`

Impacted Files	Coverage Δ
cli/autostart.go	`75.29% <100.00%> (+0.29%)`	⬆️
cli/configssh.go	`69.06% <100.00%> (+0.45%)`	⬆️
cli/create.go	`56.95% <100.00%> (+0.28%)`	⬆️
cli/delete.go	`25.00% <100.00%> (+5.76%)`	⬆️
cli/list.go	`9.52% <100.00%> (+2.96%)`	⬆️
cli/login.go	`55.85% <100.00%> (+0.23%)`	⬆️
cli/publickey.go	`76.92% <100.00%> (+0.92%)`	⬆️
cli/root.go	`79.43% <100.00%> (+0.29%)`	⬆️
cli/server.go	`57.58% <100.00%> (+0.07%)`	⬆️
cli/show.go	`23.07% <100.00%> (+6.41%)`	⬆️
... and15 more

Continue to review full report at Codecov.

Legend -Click here to learn more
Δ = absolute <relative> (impact),ø = not affected,? = missing data
Powered byCodecov. Last updatef5693df...8ea93b2. Read thecomment docs.