Skip to content

Conversation

@llucax
Copy link
Contributor

@llucax llucax commented Jun 5, 2025

Reorganize the release notes to move all "breaking changes" to the Upgrading section, and to group changes by type (removal, rename, renumbering and addition) and by protobuf package.

This hopefully will make easier for users to know how to deal with changes and what new fetures are available, depending on which packages they use/they are interested in.

Also add missing note about protobuf/grpcio dependency bump.

llucax added 2 commits June 5, 2025 10:15
This commit reorganizes the release notes to move all "breaking changes"
to the Upgrading section, and to group changes by type (removal, rename,
renumbering and addition) and by protobuf package.

This hopefully will make easier for users to know how to deal with
changes and what new fetures are available, depending on which packages
they use/they are interested in.

Signed-off-by: Leandro Lucarella <[email protected]>
Copilot AI review requested due to automatic review settings June 5, 2025 08:17
@llucax llucax requested a review from a team as a code owner June 5, 2025 08:17
@github-actions github-actions bot added the part:docs Affects the documentation label Jun 5, 2025
@llucax llucax self-assigned this Jun 5, 2025
@llucax llucax added this to the v0.7.0 milestone Jun 5, 2025
@llucax llucax enabled auto-merge June 5, 2025 08:18
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR restructures the release notes by moving breaking changes into a clear Upgrading section, grouping changes by type and protobuf package, and adds a note about the bumped protobuf/grpcio dependency.

  • Reorganized Upgrading into Removed, Renamed, Renumbered, and Dependency bump subsections
  • Refactored New Features into package-scoped groupings
  • Added missing note on protobuf/grpcio minimum version bump
Comments suppressed due to low confidence (2)

RELEASE_NOTES.md:34

  • The New Features section refers to this field as info.grid_connection_point, but here it's kind.grid_connection_point. Please standardize on one identifier to avoid confusion.
`ComponentCategoryMetadataVariant.metadata.grid` to `ElectricalComponentCategorySpecificInfo.kind.grid_connection_point`

RELEASE_NOTES.md:83

  • [nitpick] The types entry currently has no description. For consistency, consider adding a colon and brief description, e.g., - types: Package with common API types (e.g., TimeIntervalFilter).
- `types`

Copy link
Contributor

@tiyash-basu-frequenz tiyash-basu-frequenz left a comment

Choose a reason for hiding this comment

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

Approved. Please feel free to merge once the branch has bee updated.

@llucax llucax added this pull request to the merge queue Jun 5, 2025
Merged via the queue into frequenz-floss:v0.x.x with commit 0e45c6d Jun 5, 2025
6 checks passed
@llucax llucax deleted the relnotes branch June 5, 2025 08:42
@llucax
Copy link
Contributor Author

llucax commented Jun 5, 2025

Whoops, should I have mentioned more explicitly that auto-merge was enabled? 😬

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

part:docs Affects the documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants