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

Add golangci-lint #502

Merged
merged 6 commits into from
Nov 14, 2021
Merged

Add golangci-lint #502

merged 6 commits into from
Nov 14, 2021

Conversation

lukashass
Copy link
Member

@lukashass lukashass commented Nov 5, 2021

Initial part of #435
Remove Makefile tasks for govet and gofmt since they are included in golangci-lint.
Additionally add goimports since it already passes.

Makefile Show resolved Hide resolved
@6543

This comment has been minimized.

@6543 6543 added this to the 0.15.0 milestone Nov 8, 2021
@6543 6543 added blocked It's ready but something external is blocking it tests related to tests or other things CI check before merge labels Nov 8, 2021
@6543 6543 enabled auto-merge (squash) November 14, 2021 19:54
@6543 6543 removed the blocked It's ready but something external is blocking it label Nov 14, 2021
@6543 6543 merged commit c28f7cb into woodpecker-ci:master Nov 14, 2021
@lukashass lukashass deleted the add-golangci-lint branch November 14, 2021 20:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tests related to tests or other things CI check before merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants