-
Notifications
You must be signed in to change notification settings - Fork 29
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 linter rules #106
Comments
Hi @nguyer, I would like to pick up this issue. Is just adding the |
Adding the |
Thanks! Sorry I missed the PR being opened for it. I'll take a look now! |
I had quite some progress on this issue. Most of the errors are fixed. Some of them which include additional configuration are ignored for now. (e.g. adding timeouts on the server). Let me know what you think |
Currently, this repo is set up to run golangci, but has no config file, so it's missing all of the rules that we apply to our other repos. We should add the
.golangci.yml
file from here: https://github.com/hyperledger/firefly/blob/main/.golangci.ymlThen we need to go through the code and address the issues that are uncovered.
The text was updated successfully, but these errors were encountered: