-
Notifications
You must be signed in to change notification settings - Fork 692
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
chore: Fix makefile linting targets #2377
Conversation
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.
Is it necessary to update .golangci.yml
?
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.
What updates would you want to make to .golangci.yml
? This PR is specific to updating the makefile linting targets, not changing anything about golangci configuration.
@go install github.com/golangci/golangci-lint/cmd/golangci-lint@$(golangci_version) | ||
@$(golangci_lint_cmd) run --timeout=10m | ||
|
||
lint-fix: |
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.
What's the difference between make lint-fix
and make format
?
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.
make lint-fix
is just running the golanci linter, while make format
is running gofumpt
on everything that's not excluded by -not -path
, and then it runs golanci linter afterwards.
(cherry picked from commit 53397fa)
(cherry picked from commit 53397fa) Co-authored-by: lg <[email protected]>
Update Makefile linting targets to be consistent with the Cosmos SDK