Skip to content

refactor(config): key microgrids by int - #128

Merged
cwasicki merged 1 commit into
frequenz-floss:v0.x.xfrom
cwasicki:int
Aug 26, 2026
Merged

refactor(config): key microgrids by int#128
cwasicki merged 1 commit into
frequenz-floss:v0.x.xfrom
cwasicki:int

Conversation

@cwasicki

Copy link
Copy Markdown
Collaborator

Normalize microgrid map keys to plain integers, matching metadata and relation IDs. Keep string keys only at TOML and serialization boundaries.

@cwasicki
cwasicki requested review from a team as code owners August 24, 2026 16:42
@cwasicki
cwasicki requested review from llucax and micaebe and removed request for a team August 24, 2026 16:42
@github-actions github-actions Bot added part:docs Affects the documentation part:tests Affects the unit, integration and performance (benchmarks) tests labels Aug 24, 2026
@cwasicki cwasicki added status:blocked Other issues must be resolved before this can be worked on and removed part:docs Affects the documentation part:tests Affects the unit, integration and performance (benchmarks) tests labels Aug 24, 2026
@cwasicki

Copy link
Copy Markdown
Collaborator Author

Blocking since still based on #127

@github-actions github-actions Bot added part:docs Affects the documentation part:tests Affects the unit, integration and performance (benchmarks) tests labels Aug 24, 2026
Normalize microgrid map keys to plain integers, matching metadata
and relation IDs. Keep string keys only at TOML and serialization
boundaries.

Signed-off-by: cwasicki <126617870+cwasicki@users.noreply.github.com>
@cwasicki
cwasicki added this pull request to the merge queue Aug 26, 2026
Merged via the queue into frequenz-floss:v0.x.x with commit 477e637 Aug 26, 2026
9 checks passed
@cwasicki
cwasicki deleted the int branch August 26, 2026 09:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

part:docs Affects the documentation part:tests Affects the unit, integration and performance (benchmarks) tests status:blocked Other issues must be resolved before this can be worked on

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants