Run pre-commit run -a

[RothAndrew]

Description

Run SKIP=golangci-lint pre-commit run -a on the whole repo and fix what breaks

Related Issue

none

Type of change

• Refactor (non-breaking change which doesn't change existing functionality)

Checklist before merging

• Tests have been added/updated as necessary (add the needs-tests label)
• Documentation has been updated as necessary (add the needs-docs label)
• An ADR has been written as necessary (add the needs-adr label) [ 1 2 3 ]
• (Optional) Changes have been linted locally with golangci-lint. (NOTE: We haven't turned on lint checks in the pipeline yet so linting may be hard if it shows a lot of lint errors in places that weren't touched by changes. Thus, linting is optional right now.)

[jeff-mccoy]

thanks @RothAndrew lgtm