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

Fix lint errors. #127

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

twotired
Copy link
Contributor

after these changes, "make lint" is successful

make lint ; echo $?

running golangci-lint
GO111MODULE=on go list -e -compiled -test=true -export=false -deps=true -find=false -tags= -- ./... > /dev/null
GO111MODULE=on /home/jwells/go/bin/golangci-lint run ./...
0

@twotired twotired force-pushed the fix-lint-errors branch 3 times, most recently from 389314f to 4868bc7 Compare August 28, 2022 16:12
@twotired
Copy link
Contributor Author

twotired commented Sep 8, 2022

@bwplotka Could you help review?

Copy link

@luisdavim luisdavim left a comment

Choose a reason for hiding this comment

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

I don't see these getting flagged by any linter.

@bwplotka
Copy link
Member

Sorry for lag. We need rebase and improvement in our golangcilint (or even removal of it), so #156 can pass

twotired pushed a commit to twotired/jiralert that referenced this pull request May 8, 2023
Signed-off-by: Jason Wells <spinmaster@gmail.com>
twotired pushed a commit to twotired/jiralert that referenced this pull request May 8, 2023
Signed-off-by: Jason Wells <spinmaster@gmail.com>
Signed-off-by: Jason Wells <spinmaster@gmail.com>
@twotired
Copy link
Contributor Author

twotired commented May 8, 2023

@bwplotka rebase complete, please review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants