cli help text updates and flags cleanup

[lkingland]

Changes

• 🎁 verbose flag uses global setting throughout
• 🧹 confirm flag added using shared visitor throughout
• 🧹 path flag added using shared visitor throughout
• 🧹 removes --version flag on root as redundant with subcommand
• 🧹 splits main help's 'Main Commands' into 'Primary Commands' and 'Development Commands' groups
• 🧹 Other general command structure cleanup

/kind cleanup

Notes

Refactor of root help text

In order to cohere more with kn, the root help text has been reworked to be
quite similar. Subcommands were not (yet) updated. Moving the lone
global flag, verbose, to subcommands assisted in this homogenization

Addition of the verbose flag directly to commands

The complexity added by having a single global (persistent) flag was not
worth the benefit of being able to elide defining it on each command.
Furthermore, when used as a plugin (to kn for example) it is easy to be
confused which set of global commands apply where. It's clearly simpler
to just addVerbose(cmd) right along with all the other flags.

Renaming setXFlag functions to addXFlag

setX usually implies that the object with context to the accessor will be
mutaed (user.setName("bob"), etc). Whereas addX resolves this ambiguity
by implying the passed argument will be mutated.

Removing the redundant --version flag

Both to be more coherent with kn, and because it seems to be the emergent
idiom for Go apps, the builtin --version flag was removed in favor of
the extant subcommand.

Ensuring --confirm and --path flags use accessor

As stated above, there is not enough flag-usage overlap between commands
to make global ("persistent") flags correct, but there are three which
are used enough to make a helper function useful: verbose, path and confirm.
These were added where missing throughout.

Creation of the Primary Commands and Development Commands groups

It is important to list the primary CRUD operations as the primary commands,
so that new users understand the high-level lifecycle of a function.
This creates that group, moving development/helper commands such as build,
run and invoke to their own group.

Release Note

Fixes issue where global settings for --verbose and --confirm were sometimes not considered
Removes the --version flag; please use the 'version' subcommand.

[codecov]

Codecov Report

Base: 60.58% // Head: 60.61% // Increases project coverage by +0.03% 🎉

Coverage data is based on head (7649948) compared to base (9c5b5a8).
Patch coverage: 87.32% of modified lines in pull request are covered.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1564      +/-   ##
==========================================
+ Coverage   60.58%   60.61%   +0.03%     
==========================================
  Files          75       76       +1     
  Lines       10340    10363      +23     
==========================================
+ Hits         6264     6282      +18     
+ Misses       3508     3502       -6     
- Partials      568      579      +11     

Flag	Coverage Δ
integration-tests	51.04% <87.32%> (-0.28%)	⬇️
unit-tests-macos-latest	49.21% <87.32%> (?)
unit-tests-ubuntu-latest	50.69% <87.32%> (?)
unit-tests-windows-latest	49.08% <87.32%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
cmd/templates/templates.go	100.00% <ø> (ø)
cmd/config.go	30.00% <57.14%> (-0.19%)	⬇️
cmd/languages.go	86.04% <75.00%> (-3.12%)	⬇️
cmd/version.go	91.89% <75.00%> (-8.11%)	⬇️
cmd/run.go	70.63% <77.77%> (-1.73%)	⬇️
cmd/config_envs.go	26.56% <78.57%> (+0.76%)	⬆️
cmd/config_labels.go	72.89% <78.57%> (-0.41%)	⬇️
cmd/config_volumes.go	27.46% <78.57%> (+1.51%)	⬆️
cmd/list.go	64.35% <80.00%> (-5.37%)	⬇️
cmd/templates.go	80.57% <80.00%> (-1.78%)	⬇️
... and 28 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

[zroubalik]

/lgtm

Great job!

/hold for others

[lance]

Looks great overall - nice clean up. I just have a small suggestion and one question about the version values.

[zroubalik]

/lgtm

Great improvement!

[knative-prow]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: lkingland, zroubalik

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [lkingland,zroubalik]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[zroubalik]

/unhold