Dev tooling updates

[inFocus7]

Description

Updates to the dev workflow

1. Move tool dependencies to own tools/ folder to separate from root dependencies.
2. Added Helm doc file generation via make charts-docs
   • There was a pre-existing README.md.gotmpl file, but it wasn't wired up to generate the README.md.
3. Updated lint workflow step to use go tool instead of GH action
4. Added tools/go.sum to dependency caches in workflows
5. Using gci within golanci-lint*

* A lot of go imports where refactored due to the gci settings:

formatters:
  settings:
    gci:
      sections:
        - standard
        - default
        - localmodule

Change Type

/kind cleanup

Changelog

NONE

Additional Notes

Verification based on workflows running successfully.

Another verification is simply running one of the tool-based make targets - such as make lint - and see it work.

[timflannagan]

QQ on crane

[timflannagan]

Do we need either of these tool deps?

[timflannagan]

We should be using the gci golangci-lint formatter instead of the direct tool dependency. That can be done in a follow-up though. You can see the commented out config already for enabling it.

[inFocus7]

Good calls, crane was unused, but I initially kept it (in case). I just removed it and the direct call to gci + crane usage.

on a relevant note: gci was originally called only modified files, but since it's now part of the golangci-lint flow it runs for all files. Some go import formatting came into my changes 👍🏼