search

savedra1 / clipse

Configurable TUI clipboard manager for Unix
MIT License
228 stars 8 forks source link

ci: add golangci-lint to test #57

Closed ccoVeille closed 2 months ago

ccoVeille commented 2 months ago

the configuration is very minimal, it would need some configuration.

But it will start reporting issues that should be addressed

ccoVeille commented 2 months ago

I would suggest merging #56, then to merge this even if the test fails

then to try to fix them

savedra1 commented 2 months ago

Thank you for your continued work here :pray: @ccoVeille

ccoVeille commented 2 months ago

Install golangci-lint locally theb

simply launch this to report the issue

golangci-lint run

ccoVeille commented 1 month ago

I started working on this project since last time we talked. https://github.com/ccoVeille/golangci-lint-config-examples

I think about adding new rules to your project

What are your thoughts?

@savedra1

savedra1 commented 1 month ago

Hi @ccoVeille I might need some help understanding this. Would this be a new actions workflow for the project that mandates linting rules etc? I would be happy to use this if so :smile:

ccoVeille commented 1 month ago

I opened the following PR, I hope it will be clearer