Skip to content

[-] fix logging of cli options names#1178

Merged
pashagolub merged 1 commit intomasterfrom
fix-log-message
Feb 4, 2026
Merged

[-] fix logging of cli options names#1178
pashagolub merged 1 commit intomasterfrom
fix-log-message

Conversation

@0xgouda
Copy link
Collaborator

@0xgouda 0xgouda commented Feb 4, 2026

  • --refresh instead of --servers-refresh-loop-seconds
  • --batching-delay instead of --batching-delay-ms

@0xgouda 0xgouda self-assigned this Feb 4, 2026
@0xgouda 0xgouda added sinks Where and how to store monitored data sources What sources and in what way to monitor labels Feb 4, 2026
@0xgouda 0xgouda requested a review from pashagolub February 4, 2026 01:20
@coveralls
Copy link

coveralls commented Feb 4, 2026

Pull Request Test Coverage Report for Build 21654919012

Details

  • 0 of 2 (0.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.1%) to 76.281%

Changes Missing Coverage Covered Lines Changed/Added Lines %
internal/cmdopts/cmdoptions.go 0 2 0.0%
Totals Coverage Status
Change from base Build 21638311375: 0.1%
Covered Lines: 4110
Relevant Lines: 5388

💛 - Coveralls

`--refresh` instead of `--servers-refresh-loop-seconds`
`--batching-delay` instead of `--batching-delay-ms`
Copy link
Collaborator

@pashagolub pashagolub left a comment

Choose a reason for hiding this comment

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

Good catch!

@pashagolub pashagolub merged commit cc47529 into master Feb 4, 2026
9 checks passed
@pashagolub pashagolub deleted the fix-log-message branch February 4, 2026 10:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

sinks Where and how to store monitored data sources What sources and in what way to monitor

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants