Repository navigation
use ConfigKeys for event bus settings - #14471
DaanHoogland wants to merge 1 commit into
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests.
Additional details and impacted files@@ Coverage Diff @@
## ghi10752-config-foundation #14471 +/- ##
=================================================================
- Coverage 20.69% 3.88% -16.82%
=================================================================
Files 6508 493 -6015
Lines 591033 42571 -548462
Branches 72448 8069 -64379
=================================================================
- Hits 122341 1653 -120688
+ Misses 455417 40687 -414730
+ Partials 13275 231 -13044
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
|
🔴 Test Coverage Grade:
|
| Metric | Value |
|---|---|
| Line coverage | 25.53% |
| Branch coverage | 19.80% |
Grade Scale
| Grade | Line Coverage | Meaning |
|---|---|---|
| 🟢 A | ≥ 80% | Excellent - this code sleeps well at night 😴 |
| 🟡 B | 60-79% | Good - almost there, don't stop now 😉 |
| 🟠 C | 40-59% | Acceptable - your code is wearing a seatbelt, but no airbags 😬 |
| 🔴 D | 20-39% | Marginal - boldly shipping where no test has gone before 🖖 |
| ⛔ F | < 20% | Failing - tests? what tests? 🔥 |
Branch coverage is shown as a secondary signal. Grade is determined by line coverage.
View full Actions run
86803e8 to
c800b90
Compare
|
This pull request has merge conflicts. Dear author, please fix the conflicts and sync your branch with the base branch. |
Part of the split of #13884 (phase out the Config enum, #10752). Co-Authored-By: Claude Opus 5.5 <[email protected]>
676645c to
7d105be
Compare



Description
Switches the event bus publishing settings (action, alert and usage events) to
ConfigKeys.Part 5 of 12 of the split of #13884 (phase out the deprecated
Configenum, #10752). Base:ghi10752-config-foundation. Independent of the other PRs based on the foundation, so it can be reviewed and merged in any order relative to them; GitHub retargets it tomainonce the foundation is merged.Size: 5 files, +11/-30.
Split overview: foundation → parallel PRs (account, vpn-vpc, snapshot-usage, event-bus, literals) + stacked core-1 … core-5 → enum-removal (last).
Types of changes
Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale
Bug Severity
Screenshots (if appropriate):
How Has This Been Tested?
How did you try to break this feature and the system with this change?
Local verification: full reactor build (checkstyle included) and the unit tests of every touched module pass on this branch; merging all PRs of the split in any order gives no conflicts and reproduces #13884 plus its fixes.
🤖 Generated with Claude Code