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.

Sign up for GitHub

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

Jump to bottom

Setup golangci-lint and fix issues #4

Closed
Tracked by #5
Pokom opened this issue Nov 27, 2023 · 0 comments · Fixed by #109
Closed
Tracked by #5

Setup golangci-lint and fix issues #4

Pokom opened this issue Nov 27, 2023 · 0 comments · Fixed by #109
Assignees
@logyball
Labels
good first issue Good for newcomers

Comments

@Pokom
Copy link
Contributor

Pokom commented Nov 27, 2023 •
edited
Loading

We don't have use any form of static analysis for the cloudcost-exporter codebase. It would be nice to setup a barebone golangci-lint configuration for us to get started. I imagine the first PR to introduce it will also require us to fix any issues.
@Pokom Pokom mentioned this issue Nov 29, 2023
Closed
@Pokom Pokom transferred this issue from another repository Nov 29, 2023
@Pokom Pokom added the good first issue Good for newcomers label Nov 30, 2023
@logyball logyball self-assigned this Feb 16, 2024
@logyball logyball mentioned this issue Feb 16, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@logyball logyball

Labels
good first issue Good for newcomers
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add Linting grafana/cloudcost-exporter
2 participants
@Pokom @logyball