Closed Sovietaced closed 1 week ago
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 60.99%. Comparing base (
de415af
) to head (40dafb4
). Report is 1 commits behind head on master.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
This pull request introduces a
lint-fix
target that will automatically fix linting issues instead of runninggoimports
,gci
, orgolangci-lint
commands manually.It also adds
gci
linting toflytectl
. It appears thatflytectl
hadgolangci-lint
config options forgci
but the linter was not enabled.Lastly, it removes the
goimports
make target which was pretty error prone. It did not respect thegolangci-lint
configs in each project and generated dramatically different fixes.