Add golangci-lint configuration and appease some linters#13
Conversation
livingsilver94
left a comment
There was a problem hiding this comment.
Your linting PR was made against master, but I reworked the cli package in the relive branch, you should lint that branch instead.
I don't understand: this PR is a standalone change that has little to do with the CLI overhaul. As such, it should generally go into master. |
|
You linted |
|
Should you not like the "temporary master branch" approach but prefer advancing the master in little steps, just lemme know and we'll go that way. My initial idea was to resume working on usysconf in relive and eventually merge it into master. |
That's fine, these changes can wait 🙂
Personally, I would reserve the 'temporary master branch' (or 'long-lived feature branch') approach for when master is actively being developed and/or changes are likely to subtly break things. That doesn't seem the case here, so I would just advance master in little steps. |
|
@livingsilver94: I have rebased on the latest master. GLHF 😉 |
This PR adds configuration and a GitHub workflow action for golangci-lint. Trivially fixed linting issues have been fixed, and other linters disabled so they can be enabled with a little more care..