-
Notifications
You must be signed in to change notification settings - Fork 439
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
add golang lint #3154
add golang lint #3154
Conversation
19e7397
to
98b78a5
Compare
0d58263
to
87f58a3
Compare
Makefile
Outdated
@@ -828,6 +828,10 @@ lint: | |||
@GOOS=linux go vet ./... | |||
@GOOS=linux gosec -exclude=G204,G306,G404,G601,G301 -exclude-dir=test -exclude-dir=pkg/client ./... | |||
|
|||
.PHONY: gofumpt | |||
gofumpt: gofumpt | |||
find . -name "*.go" -execdir gofumpt -w -extra {} \; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这里用 -w 重写了,那 lint 的问题是不是就检查不出来了,但是 git 仓库里还没改
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
是的 ,这个步骤不该放流水线里面。可以手动执行。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
a601e23
to
62c767f
Compare
62c767f
to
a69b068
Compare
What type of this PR
Examples of user facing changes:
Which issue(s) this PR fixes:
Fixes #(issue-number)
WHAT
🤖 Generated by Copilot at ec95eea
Improved the linting workflow and configuration for the project. Added a
.github/workflows/.golangci.yml
file to customize the golangci-lint tool and modified the.github/workflows/lint.yaml
file to use the new configuration and filter the relevant files.🤖 Generated by Copilot at ec95eea
HOW
🤖 Generated by Copilot at ec95eea
.golangci.yml
file to configure the golangci-lint tool for the project (link).golangci.yml
file is changed (link)