ci: Add golangci-lint configuration and fix initial linting errors - Githubissues

supitsdu / clipper

Seamlessly copy file contents to clipboard from command line. Lightweight, cross-platform tool for instant text transfers.

MIT License

3 stars 3 forks source link

ci: Add golangci-lint configuration and fix initial linting errors #32

Closed ccoVeille closed 2 months ago

ccoVeille commented 2 months ago

This pull request lays the groundwork for improving code quality by integrating golangci-lint, a powerful Go linter. Here's what this PR accomplishes:

Configuration Setup:
- Adds the .golangci.yml configuration file.
- This configuration ensures consistent code style and helps identify potential issues.
Initial Code Cleanup:
- Fixes linting errors detected by golangci-lint in our existing codebase.
- These corrections address stylistic inconsistencies and minor code smells.

Next Steps (Not Included in This PR):

CI Integration: The next step is to integrate golangci-lint into our CI pipeline. This will enable automated linting on each pull request.
Additional Rules: As we gain experience with golangci-lint, we can gradually enable more rules to enhance our code quality further.

Review Checklist:

[x] Verify that golangci-lint configuration is appropriate for our project.
[x] All initial linting errors have been adequately addressed.
[x] Confirm that the changes do not introduce regressions or unexpected behavior.

Fixes #20

ccoVeille commented 2 months ago

Yes, it can be added very easily.

Have fun with golangci-lint and revive. It's a sane journey