commands: Use consistent style when describing subcommands

[Hasaber8]

Adapting to language suggested in - (#12897)

Change-Id: Ib27a4a7b540d45243e6252db769d4b9fb7447718

[CLAassistant]

All committers have signed the CLA.

[bep]

The CLI docs is generated with go run main.go gen doc --dir docs/content/en/commands, something we typically do before we release.

As to this PR, it would be much easier to review it without all the docs changes. So could you revert the docs folder to master and force push the PR branch?

[Hasaber8]

Done!

[bep]

This looks very good, but there are some failing test assertions.

[Hasaber8]

This looks very good, but there are some failing test assertions.

Updated.

[jmooring]

There are two remaining subcommands to address: version and build.

Current (with this PR):

Available Commands:
  build       build builds your site
  ...
  version     Display version and environment info

As suggested in #12897:

Available Commands:
  build       Build your site
  ...
  version     Display version

[Hasaber8]

There are two remaining subcommands to address: version and build.

Current (with this PR):

Available Commands:
  build       build builds your site
  ...
  version     Display version and environment info

As suggested in #12897:

Available Commands:
  build       Build your site
  ...
  version     Display version

Updated!

[jmooring]

Now we have this:

Available Commands:
  build       Build your site
  ...
  env         Display version
  ...
  version     Display version

The env description should be "Display version and environment info". The env and version subcommands do different things.

[Hasaber8]

@jmooring made the requested changes.

[jmooring]

This still isn't correct. See my previous comment regarding "version" vs. "env".

[Hasaber8]

This still isn't correct. See my previous comment regarding "version" vs. "env".

I misread that one, its updated now.

[jmooring]

You have at least one failing test. Run mage -v test locally.

[Hasaber8]

You have at least one failing test. Run mage -v test locally.

Fixed.

[jmooring]

This looks good. Please squash the commits, change the commit message to "Closes #12897", and force push.

[Hasaber8]

This looks good. Please squash the commits, change the commit message to "Closes #12897", and force push.

All squashed!

[jmooring]

This is a nit, but please change this:

- Closes #12897

To what I asked for in my previous comment:

Closes #12897

See https://github.com/gohugoio/hugo/blob/master/CONTRIBUTING.md#git-commit-message-guidelines.

[Hasaber8]

This is a nit, but please change this:

- Closes #12897

To what I asked for in my previous comment:

Closes #12897

See https://github.com/gohugoio/hugo/blob/master/CONTRIBUTING.md#git-commit-message-guidelines.

Done, will keep this in mind for future contributions.

Thank you very much for the tool.

[jmooring]

Current commit message:

Closes (#12897)

What I asked for:

Closes #12897