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.

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

cognitive complexity 32 of func (*ExitState).ReturnCheckResults is high (> 10) (gocognit) #127

Closed
atc0005 opened this issue Jun 16, 2022 · 0 comments · Fixed by #128
Closed

cognitive complexity 32 of func (*ExitState).ReturnCheckResults is high (> 10) (gocognit) #127

atc0005 opened this issue Jun 16, 2022 · 0 comments · Fixed by #128
Assignees
@atc0005
Labels
bug Something isn't working linting
Milestone
v0.9.0

Comments

@atc0005
Copy link
Owner

atc0005 commented Jun 16, 2022

Output from a recent unstable go-ci container run:

nagios.go:290:1: cognitive complexity 32 of func `(*ExitState).ReturnCheckResults` is high (> 10) (gocognit)
func (es *ExitState) ReturnCheckResults() {
^
@atc0005 atc0005 added bug Something isn't working linting labels Jun 16, 2022
@atc0005 atc0005 added this to the Next Release milestone Jun 16, 2022
@atc0005 atc0005 self-assigned this Jun 16, 2022
atc0005 added a commit that referenced this issue Jun 16, 2022
@atc0005
Split ExitState.ReturnCheckResults() into helpers
00cbd3b 
Satisfy gocognit linter, but also (hopefully) make the code
easier to maintain going forward.

refs GH-127
atc0005 added a commit that referenced this issue Jun 16, 2022
@atc0005
Split ExitState.ReturnCheckResults() into helpers
ce87db3 
Satisfy gocognit linter, but also (hopefully) make the code
easier to maintain going forward.

refs GH-127
@atc0005 atc0005 mentioned this issue Jun 16, 2022
Merged
atc0005 added a commit that referenced this issue Jun 16, 2022
@atc0005
Split ExitState.ReturnCheckResults() into helpers
f939905 
Satisfy gocognit linter, but also (hopefully) make the code
easier to maintain going forward.

refs GH-127
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@atc0005 atc0005

Labels
bug Something isn't working linting
Projects
None yet
Milestone
v0.9.0
Development

Successfully merging a pull request may close this issue.

Split ExitState.ReturnCheckResults() into helpers atc0005/go-nagios
1 participant
@atc0005