update hack/tools, update golangci-lint and fix lints

liquidmetal-dev / flintlock

Lock, Stock, and Two Smoking MicroVMs. Create and manage the lifecycle of MicroVMs backed by containerd.

Mozilla Public License 2.0

529 stars 32 forks source link

update hack/tools, update golangci-lint and fix lints #879

Closed cpanato closed 2 months ago

cpanato commented 2 months ago

What this PR does / why we need it:

update hack/tools, update golangci-lint and fix lints

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged): Fixes #

Special notes for your reviewer:

Checklist:

[x] squashed commits into logical changes
[ ] includes documentation
[ ] adds unit tests
[ ] adds or updates e2e tests

cpanato commented 2 months ago

cc @richardcase

I'm Working to fix the lints. I might ignore some for now to not have a super huge PR; is that ok?

richardcase commented 2 months ago

I'm Working to fix the lints. I might ignore some for now to not have a super huge PR; is that ok?

Sounds like a plan. We can circle back to them in the future.

richardcase commented 2 months ago

Thanks for this @cpanato, the time and effort is very much appreciated 🎉