Skip to content

chore: added golangci linter to improve code quality#40

Merged
jay-dee7 merged 1 commit intomasterfrom
linter
Oct 1, 2021
Merged

chore: added golangci linter to improve code quality#40
jay-dee7 merged 1 commit intomasterfrom
linter

Conversation

@guacamole
Copy link
Member

Signed-off-by: guacamole [email protected]

  • Enabled following linters apart from default:
    • bodyclose
    • cyclop
    • gochecknoglobals
    • goimports
    • gosec
    • lll
    • nilerr
    • noctx
    • sqlclosecheck
    • staticcheck

@guacamole guacamole requested a review from jay-dee7 October 1, 2021 02:50
@guacamole guacamole self-assigned this Oct 1, 2021
Copy link
Member

@jay-dee7 jay-dee7 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🥇

@jay-dee7 jay-dee7 merged commit 8aebc63 into master Oct 1, 2021
@jay-dee7 jay-dee7 deleted the linter branch October 1, 2021 03:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants