Skip to content

Conversation

@tiyash-basu-frequenz
Copy link
Contributor

No description provided.

@tiyash-basu-frequenz tiyash-basu-frequenz added this to the v0.8.0 milestone Jul 10, 2025
@tiyash-basu-frequenz tiyash-basu-frequenz self-assigned this Jul 10, 2025
Copilot AI review requested due to automatic review settings July 10, 2025 09:03
@tiyash-basu-frequenz tiyash-basu-frequenz requested a review from a team as a code owner July 10, 2025 09:03
@github-actions github-actions bot added the part:docs Affects the documentation label Jul 10, 2025
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

The PR cleans up and reformats the RELEASE_NOTES.md for the v1alpha8 release, reorganizing the breaking changes into a clearer structure.

  • Updated the summary to introduce v1alpha8 and highlight major breaking changes.
  • Refactored the Upgrading section to list removed components and renamed symbols.
  • Consolidated change categories under a single coherent layout.
Comments suppressed due to low confidence (3)

RELEASE_NOTES.md:7

  • [nitpick] Consider adding a subheading like '### Breaking Changes' before the lists of removed and renamed items to clearly separate upgrade instructions from other content.
## Upgrading

RELEASE_NOTES.md:11

  • [nitpick] Rather than using a list item for 'Removed:', consider using a markdown subheading (e.g., '#### Removed') or bold text to improve readability of the breaking changes section.
- Removed:

RELEASE_NOTES.md:13

  • [nitpick] The nested list marker '+' mixed with '-' and '' in other parts can be confusing; consider using a consistent marker (e.g., all '-' or all '') throughout.
    + `electrical_components.Fuse`

Copy link
Contributor

@thomas-nicolai-frequenz thomas-nicolai-frequenz left a comment

Choose a reason for hiding this comment

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

LGTM

@tiyash-basu-frequenz tiyash-basu-frequenz added this pull request to the merge queue Jul 10, 2025
Merged via the queue into frequenz-floss:v0.x.x with commit 2678cee Jul 10, 2025
6 checks passed
@tiyash-basu-frequenz tiyash-basu-frequenz deleted the revamp_release_notes branch July 10, 2025 09:35
* `ElectricalComponentCategorySpecificInfo.kind.voltage_transformer` to `ElectricalComponentCategorySpecificInfo.kind.power_transformer`
* `ElectricalComponentCategory.ELECTRICAL_COMPONENT_CATEGORY_VOLTAGE_TRANSFORMER` to `ElectricalComponentCategory.ELECTRICAL_COMPONENT_CATEGORY_POWER_TRANSFORMER`

+ `types`:
Copy link
Contributor

Choose a reason for hiding this comment

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

This should probably be type, we can fix it in the GitHub release notes at least.

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.

3 participants