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 "make lint" target and add to "make test" target #280

Closed
bnevis-i opened this issue Oct 6, 2021 · 0 comments · Fixed by #302
Closed

Add "make lint" target and add to "make test" target #280

bnevis-i opened this issue Oct 6, 2021 · 0 comments · Fixed by #302
Assignees
Labels
1-low priority denoting isolated changes tech-debt issue_type denoting refactoring to improve design or removal of temporary workarounds
Milestone

Comments

@bnevis-i
Copy link
Collaborator

bnevis-i commented Oct 6, 2021

Should enable golangci-lint with default linters + gosec.

See edgexfoundry/edgex-go#3565

@lenny-goodell lenny-goodell added 1-low priority denoting isolated changes tech-debt issue_type denoting refactoring to improve design or removal of temporary workarounds labels Oct 14, 2021
@lenny-goodell lenny-goodell added this to the Jakarta milestone Oct 14, 2021
@bnevis-i bnevis-i modified the milestones: Jakarta, Kamakura Nov 8, 2021
@bnevis-i bnevis-i linked a pull request Jan 11, 2022 that will close this issue
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1-low priority denoting isolated changes tech-debt issue_type denoting refactoring to improve design or removal of temporary workarounds
Projects
None yet
3 participants