Skip to content

Conversation

@SlimaneAmar
Copy link
Collaborator

Please check if the PR fulfills these requirements

  • [ x] The commit message follows our guidelines
  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)
  • A PR or issue has been opened in all impacted repositories (if any)

Does this PR already have an issue describing the problem?

No

What kind of change does this PR introduce?

Feature

What is the current behavior?

The new legend writer is not used

What is the new behavior (if this is a feature change)?
The new legend writer is used

Does this PR introduce a breaking change or deprecate an API?

  • Yes
  • [x ] No

Slimane AMAR added 3 commits November 14, 2025 17:19
Copy link
Contributor

@klesaulnier klesaulnier left a comment

Choose a reason for hiding this comment

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

removed unused method
Code: OK
Test: OK

return value + " " + UNIT_MW;
}

private static OptionalDouble sumDoubleStream(DoubleStream stream) {
Copy link
Contributor

Choose a reason for hiding this comment

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

unused, to remove

Slimane AMAR added 4 commits December 5, 2025 11:13
Signed-off-by: Slimane AMAR <[email protected]>
…-diagram-server into add_sld_lengend_writer

Signed-off-by: Slimane AMAR <[email protected]>

# Conflicts:
#	src/main/java/com/powsybl/sld/server/SingleLineDiagramService.java
Signed-off-by: Slimane AMAR <[email protected]>
@sonarqubecloud
Copy link

sonarqubecloud bot commented Dec 9, 2025

@SlimaneAmar SlimaneAmar merged commit d932956 into main Dec 9, 2025
5 checks passed
@SlimaneAmar SlimaneAmar deleted the add_sld_lengend_writer branch December 9, 2025 16:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants