Skip to content

Link class and Branch class derive from new Edge class - #1546

Open
Jerry-Jinfeng-Guo wants to merge 13 commits into
mainfrom
pgm/feature/register-links-for-merging
Open

Link class and Branch class derive from new Edge class#1546
Jerry-Jinfeng-Guo wants to merge 13 commits into
mainfrom
pgm/feature/register-links-for-merging

Conversation

@Jerry-Jinfeng-Guo

@Jerry-Jinfeng-Guo Jerry-Jinfeng-Guo commented Aug 22, 2026

Copy link
Copy Markdown
Member

This PR adds the registration change to component Link:

  • New Edge class as base class for both Link and Branch
  • Existing tests should pass without alternation
  • New unit tests for Edge class and new deriving relationship

Registration of Link under different conditions to different code paths is scoped out and will come in a follow up PR.

mgovers and others added 2 commits August 18, 2026 16:06
Signed-off-by: Martijn Govers <Martijn.Govers@Alliander.com>
…ent and output etc

Signed-off-by: Jerry Jinfeng Guo <jerry.jinfeng.guo@alliander.com>
@Jerry-Jinfeng-Guo Jerry-Jinfeng-Guo self-assigned this Aug 22, 2026
@Jerry-Jinfeng-Guo Jerry-Jinfeng-Guo added feature New feature or request do-not-merge This should not be merged labels Aug 22, 2026
Jerry-Jinfeng-Guo and others added 11 commits August 22, 2026 12:01
Signed-off-by: Jerry Guo <jerryguo@rocinantemac.home>
I, Jerry Guo <guojinfeng8621@gmail.com>, hereby add my Signed-off-by to this commit: a6a6355

Signed-off-by: Jerry Guo <guojinfeng8621@gmail.com>
Signed-off-by: Jerry Guo <guojinfeng8621@gmail.com>
Signed-off-by: Jerry Guo <guojinfeng8621@gmail.com>
Signed-off-by: Jerry Guo <guojinfeng8621@gmail.com>
Signed-off-by: Jerry Guo <guojinfeng8621@gmail.com>
Signed-off-by: Jerry Guo <guojinfeng8621@gmail.com>
Signed-off-by: Jerry Guo <guojinfeng8621@gmail.com>
Signed-off-by: Jerry Guo <guojinfeng8621@gmail.com>
Signed-off-by: Jerry Guo <guojinfeng8621@gmail.com>
@Jerry-Jinfeng-Guo
Jerry-Jinfeng-Guo marked this pull request as ready for review August 23, 2026 12:55
@Jerry-Jinfeng-Guo Jerry-Jinfeng-Guo removed the do-not-merge This should not be merged label Aug 23, 2026
@sonarqubecloud

Copy link
Copy Markdown

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

Labels

feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants