Skip to content

Remove replace_unspecified_code_type_with#253

Merged
llucax merged 1 commit into
frequenz-floss:v0.x.xfrom
llucax:revert-default
Jul 17, 2026
Merged

Remove replace_unspecified_code_type_with#253
llucax merged 1 commit into
frequenz-floss:v0.x.xfrom
llucax:revert-default

Conversation

@llucax

@llucax llucax commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

This new keyword-only parameter was added to delivery_area_from_proto2() because the system was not setting code_type properly always. This has been fixed so this hack is not needed anymore.

Refs #241.

@llucax
llucax requested a review from a team as a code owner July 17, 2026 07:54
@llucax
llucax requested review from shsms and removed request for a team July 17, 2026 07:54
@github-actions github-actions Bot added part:tests Affects the unit, integration and performance (benchmarks) tests part:grid Affects the grid protobuf definitions labels Jul 17, 2026
@llucax
llucax requested a review from cwasicki July 17, 2026 09:32
@llucax llucax added this to the v0.4.1 milestone Jul 17, 2026
@llucax llucax added the cmd:skip-release-notes It is not necessary to update release notes for this PR label Jul 17, 2026
@llucax llucax self-assigned this Jul 17, 2026
@llucax
llucax enabled auto-merge July 17, 2026 09:33
This new keyword-only parameter was added to
`delivery_area_from_proto2()` because the system was not setting
`code_type` properly always. This has been fixed so this hack is not
needed anymore.

Signed-off-by: Leandro Lucarella <luca-frequenz@llucax.com>
@llucax
llucax added this pull request to the merge queue Jul 17, 2026
Merged via the queue into frequenz-floss:v0.x.x with commit a8df5ae Jul 17, 2026
9 checks passed
@llucax
llucax deleted the revert-default branch July 17, 2026 09:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cmd:skip-release-notes It is not necessary to update release notes for this PR part:grid Affects the grid protobuf definitions part:tests Affects the unit, integration and performance (benchmarks) tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants