Skip to content
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

Open
nguyer opened this issue Feb 17, 2023 · 6 comments
Open

Add linter rules #106

nguyer opened this issue Feb 17, 2023 · 6 comments
Assignees
Labels

Comments

@nguyer
Copy link
Contributor

nguyer commented Feb 17, 2023

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.yml

Then we need to go through the code and address the issues that are uncovered.

@PanGan21
Copy link
Contributor

PanGan21 commented Mar 8, 2023

Hi @nguyer, I would like to pick up this issue. Is just adding the .golangci.yml sufficient enough?

@nguyer
Copy link
Contributor Author

nguyer commented Mar 8, 2023

Adding the .golangci.yml is the first step. Then make will start failing because the code doesn't meet the requirements of the linter rules in various places. So the second part of this issue is fixing those issues so that make succeeds again.

@PanGan21
Copy link
Contributor

Hi @nguyer I think both subtasks are done. I would appreciate a first review for #108 :)

@nguyer
Copy link
Contributor Author

nguyer commented Mar 15, 2023

Thanks! Sorry I missed the PR being opened for it. I'll take a look now!

@PanGan21
Copy link
Contributor

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

@tobigiwa
Copy link

@nguyer is this issue still open? or how is progress on it @PanGan21 ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants