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

feat: add golangci-lint #165

Merged
merged 1 commit into from
Jul 15, 2024
Merged

feat: add golangci-lint #165

merged 1 commit into from
Jul 15, 2024

Conversation

uhthomas
Copy link
Collaborator

@uhthomas uhthomas commented May 9, 2024

golangci-lint runs many analysers, including staticcheck, which prevent correctness issues and catch bugs.

https://golangci-lint.run/

Fixes: #160

@uhthomas uhthomas force-pushed the 160 branch 4 times, most recently from 6392c4e to 376db83 Compare May 9, 2024 14:13
@uhthomas

This comment was marked as resolved.

@uhthomas uhthomas changed the base branch from main to 160-staticcheck May 9, 2024 14:55
@uhthomas uhthomas force-pushed the 160 branch 3 times, most recently from 0ca8176 to 362d828 Compare May 9, 2024 21:48
@uhthomas uhthomas changed the base branch from 160-staticcheck to main May 9, 2024 21:48
@uhthomas uhthomas force-pushed the 160 branch 3 times, most recently from be9479c to 2b43235 Compare May 9, 2024 22:08
@uhthomas
Copy link
Collaborator Author

uhthomas commented Jul 7, 2024

@jodevsa Would it be possible to merge this soon? Now? It looks like a lot of changes are being made and it would be great to prevent potential issues!

@uhthomas uhthomas requested a review from jodevsa July 7, 2024 19:17
@winston0410
Copy link
Collaborator

Hey @uhthomas I am willing to take on this with you. Checked the PR and it looks good, do you mind resolving the conflict, so we can merge this?

@winston0410 winston0410 merged commit 48cb13b into main Jul 15, 2024
6 checks passed
@jodevsa
Copy link
Owner

jodevsa commented Jul 15, 2024

🎉 This issue has been resolved in version 2.5.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

linting?
3 participants