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.

Sign up for GitHub

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

Jump to bottom

Add golangci-lint and gosec to Makefile #18

Merged

jgwest merged 1 commit into argoproj-labs:main from jgwest:add-gosec-golangcilint

Jan 17, 2024

Member

jgwest commented Jan 16, 2024 •

edited

Loading

This PR:

Adds golangci-lint and gosec to Makefile
Update GitHub workflow to call them (rather than hardcoding the version in the workflow file)
Fixes a few minor issues reported by the tools

jgwest force-pushed the add-gosec-golangcilint branch from 237d953 to cc0bb61 Compare

January 16, 2024 08:54


          Add golangci-lint and gosec to Makefile

1ed440b

Signed-off-by: Jonathan West <[email protected]>

jgwest force-pushed the add-gosec-golangcilint branch from cc0bb61 to 1ed440b Compare

January 16, 2024 08:59

jgwest merged commit cf4e239 into argoproj-labs:main

3 checks passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet