Skip to content

Conversation

@adombeck
Copy link
Contributor

@adombeck adombeck commented Oct 15, 2025

WIP because I didn't test it yet

We might also want to use a map for defining the sections and keys so that we don't have to keep the list of variables in sync with the new map.

TODO

  • add tests

WIP because I didn't test it yet

We might also want to use a map for defining the sections and keys so
that we don't have to keep the list of variables in sync with the new
map.
@adombeck adombeck changed the title WIP: Warn about unused settings WIP: Warn about unused settings in broker.conf Oct 15, 2025
@codecov-commenter
Copy link

Codecov Report

❌ Patch coverage is 77.77778% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 82.10%. Comparing base (49ac33a) to head (cbbfdd2).

Files with missing lines Patch % Lines
internal/broker/config.go 77.77% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #706      +/-   ##
==========================================
- Coverage   82.14%   82.10%   -0.04%     
==========================================
  Files          21       21              
  Lines         997     1006       +9     
==========================================
+ Hits          819      826       +7     
- Misses        178      180       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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.

2 participants