chartmuseum
chartmuseum copied to clipboard
Add linting steps to CI (github actions)
We should introduce linting checks as a part of our CI process that are ran on both PRs and the main branch.
-
- [x] Add a golangci-lint Make target
-
- [x] Resolve linting failures that currently exist (either fix or add skips)
-
- [ ] Add a linting step to github actions that runs on PRs and main. The official golangci-lint action will comment on PRs making it easier to surface issues, that might be the way to go.
The linting failures can be worked overtime as long as they are all resolved before linting is added to GitHub actions.
Hey, @cbuto I would like to pick up this issue.
Sure please, go ahead and pick it up. I was supposed to work on it but my hands are full right now. Let us follow if you have any follow up questions
Awesome, Thanks @nerdeveloper. you can also assign the issue to me.