Skip to content

Add LfNetwork copy. - #1433

Open
gautierbureau wants to merge 4 commits into
mainfrom
lfnetwork_copy
Open

Add LfNetwork copy.#1433
gautierbureau wants to merge 4 commits into
mainfrom
lfnetwork_copy

Conversation

@gautierbureau

@gautierbureau gautierbureau commented Jun 18, 2026

Copy link
Copy Markdown
Member

Please check if the PR fulfills these requirements

  • 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?

What kind of change does this PR introduce?

It introduces a copy constructor for LfNetwork

What is the current behavior?

LfNetwork is not copiable

What is the new behavior (if this is a feature change)?
LfNetwork can be copied

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

  • Yes
  • No

If yes, please check if the following requirements are fulfilled

  • The Breaking Change or Deprecated label has been added
  • The migration steps are described in the following section

What changes might users need to make in their application due to this PR? (migration steps)

Other information:

@gautierbureau
gautierbureau requested a review from geofjamg June 18, 2026 09:35
@gautierbureau
gautierbureau marked this pull request as draft June 18, 2026 09:35
@gautierbureau
gautierbureau force-pushed the lfnetwork_copy branch 5 times, most recently from a4df436 to cc28014 Compare June 19, 2026 06:53
@gautierbureau
gautierbureau marked this pull request as ready for review June 22, 2026 08:31
Signed-off-by: Gautier Bureau <gautier.bureau@rte-france.com>
Signed-off-by: Sylvestre Prabakaran <sylvestre.prabakaran@rte-france.com>
Signed-off-by: Sylvestre Prabakaran <sylvestre.prabakaran@rte-france.com>
Signed-off-by: Sylvestre Prabakaran <sylvestre.prabakaran@rte-france.com>
@sonarqubecloud

Copy link
Copy Markdown

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants