helm-charts icon indicating copy to clipboard operation
helm-charts copied to clipboard

Run `go fmt` or `gofumpt` against go code

Open chrisseto opened this issue 10 months ago • 2 comments

We don't currently have any style enforcement of our go code. Let's start running go fmt or gofumpt within CI to enforce consistent styling and fix the occasional diff churn with all the +gotohelm annotations/markers.

chrisseto avatar Apr 15 '24 15:04 chrisseto

It has been a while since i made changes to our CI, would like to use this ticket as an excuse to checkout out the current changes and start contributing here. You ok with this @chrisseto ?

alejandroEsc avatar Apr 16 '24 14:04 alejandroEsc

Sure but let's try to work through our planned queue first? We can toss this in and push it a little bit up the stack but I'd like us to make a bit more progress on the go conversions.

chrisseto avatar Apr 16 '24 15:04 chrisseto