-
Notifications
You must be signed in to change notification settings - Fork 822
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
linters: Bump golangci-lint version to v1.52.2 and fix issues #1169
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.
Always nice to see you get more familiar with our codebase 🎉
You'll just need to be more specific with your nolints
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.
Thanks for the changes! Are you getting these linter issues from the most recent version? If so, can you please bump all versions for GitHub actions like linter.yml
and else ware?
Yes, I found these issues after upgrading my linter. golangci-lint v1.52.2 with go1.20.2. Okay sir I will update the .github/workflows/linter.yml file with my next commit. After few minutes |
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.
This is lovely. Thanks! 👍
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.
tACK! Thanks for making those changes
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.
Nice work @samuael !
Thank you, sir. |
This pull request contains several fixes to the gocryptotrader codebase, including a fix for a linter issue, suppressing some linter issues by adding a linter directive, removing unnecessary linter directives, and changing some variable names to constants.
Fixes # (issue)
Type of change
Please delete options that are not relevant and add an
x
in[]
as the item is complete.How has this been tested
Checklist