matrixji opened a new pull request, #2084: URL: https://github.com/apache/incubator-devlake/pull/2084
- run golangci-lint in lake-builder to ensure the toolchain is same as builder - fix some header missing errors - add //nolint and TODO for other errors(4 in total), which we may handle later - enable gofmt in golangci-lint and add `make fmt` for format the whole project, this could keep us always with the same style. fixes: #2077 Signed-off-by: Ji Bin <[email protected]> ### ⚠️ Pre Checklist > Please complete _ALL_ items in this checklist, and remove before submitting - [x] I have read through the [Contributing](https://devlake.apache.org/community/) Documentation - [x] This PR is using a `label` (bug, feature etc.) - [ ] My code is has necessary documentation (if appropriate) - [ ] I have added any relevant tests - [x] This section (**⚠️ Pre Checklist**) will be removed when submitting PR # Summary <!-- Thanks for submitting a pull request! We appreciate you spending the time to work on these changes. Please fill out as many sections below as possible. --> ### Does this close any open issues? #2077 ### Screenshots After this change, the lint jobs got: ``` Running [/github/home/golangci-lint-1.43.0-linux-amd64/golangci-lint run --out-format=github-actions] in [] ... golangci-lint found no issues Ran golangci-lint in 119980ms ``` and previously: ``` Running [/home/runner/golangci-lint-1.43.0-linux-amd64/golangci-lint run --out-format=github-actions] in [] ... level=warning msg="[runner] Can't run linter goanalysis_metalinter: buildir: failed to load package git: could not load export data: no export data for \"github.com/libgit2/git2go/v33\"" golangci-lint found no issues Ran golangci-lint in 20615ms ``` ### Other Information N/A -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
