Skip to content

Fix golangci-lint#813

Merged
AlexStocks merged 9 commits intoapache:mainfrom
hokkine:fix-golangci-lint
Mar 16, 2025
Merged

Fix golangci-lint#813
AlexStocks merged 9 commits intoapache:mainfrom
hokkine:fix-golangci-lint

Conversation

@hokkine
Copy link
Contributor

@hokkine hokkine commented Mar 13, 2025

What this PR does:

Upgrade the golangci-lint version to 1.61 and modify the linter configuration.

Which issue(s) this PR fixes:

N/A

Special notes for your reviewer:

Does this PR introduce a user-facing change?:

NONE

@hokkine
Copy link
Contributor Author

hokkine commented Mar 14, 2025

I have already fixed the golangci-lint script, but the CI script still fails. After troubleshooting, I found that running the go fmt ./... command locally revealed a large amount of unformatted code. The CI script performs a Go formatting check, which causes it to fail (maybe).

@FoghostCn
Copy link
Contributor

I have already fixed the golangci-lint script, but the CI script still fails. After troubleshooting, I found that running the go fmt ./... command locally revealed a large amount of unformatted code. The CI script performs a Go formatting check, which causes it to fail (maybe).

just format go.mod will work

@AlexStocks AlexStocks merged commit a633856 into apache:main Mar 16, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants