Graylog2 / Graylog2/collector

Combine error checks

Open
#6 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Go
Stars
1
Forks
0
Avg merge
3d 1h
Merged PRs (30d)
7

Description

https://github.com/Graylog2/collector-sidecar/blob/v2/superv/supervisor/supervisor.go#L785-L791

Since both of these need to be set, it would be nice if the error checking/reporting was also combined and not one after the other.

Contributor guide

Open the contributing guide

Research direction

Open superv/supervisor/supervisor.go at lines 785-791 and inspect the two error checks and their reporting. Confirm how both required values are checked, then combine the checks and run the relevant project tests. Done means the required checks are reported together rather than one after the other.

Written by the indexing model from the issue text.

Assessment

Tech stack
go
Domain
devops
Issue type
Refactor
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Quiet
Clarity
Clearly specified
Newbie friendliness
55/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.