Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Retire ControlFlow services #471

Closed
prmr opened this issue Aug 11, 2022 · 0 comments
Closed

Retire ControlFlow services #471

prmr opened this issue Aug 11, 2022 · 0 comments
Assignees
@prmr
Labels
main Issues with the main code base refactoring A proposal to refector or generally improve the code
Milestone
Release 3.5

Comments

@prmr
Copy link
Owner

prmr commented Aug 11, 2022

Retire the control flow class and replace it with the same services offered via the SequenceDiagramRenderer.
@prmr prmr added refactoring A proposal to refector or generally improve the code main Issues with the main code base labels Aug 11, 2022
@prmr prmr added this to the Release 3.5 milestone Aug 11, 2022
@prmr prmr self-assigned this Aug 11, 2022
prmr added a commit that referenced this issue Sep 21, 2022
@prmr
Issue #471 Remove currently unused code
032c1b0
prmr added a commit that referenced this issue Sep 21, 2022
@prmr
Issue #471 Replace ControlFlow#getCallees with Renderer version
3918ce4
prmr added a commit that referenced this issue Sep 21, 2022
@prmr
Issue #471 Move hasEntryPoint to Renderer
1a040c8
prmr added a commit that referenced this issue Sep 21, 2022
@prmr
Issue #471 Move getCorrespondingReturnEdge to Builder
e860b76
prmr added a commit that referenced this issue Sep 21, 2022
@prmr
Issue #471 Move getEdgeStart to renderer
918c88f
prmr added a commit that referenced this issue Sep 27, 2022
@prmr
Issue #471 Move getNodeUpstreams to SequenceDiagramBuilder
587a0e6
prmr added a commit that referenced this issue Sep 27, 2022
@prmr
Issue #471 Completely retire the ControlFlow class
014e3c9
Laurieprince added a commit to Laurieprince/JetUML that referenced this issue Oct 21, 2022
@Laurieprince
Issue prmr#471 Implement tests for SequenceDiagram builder and renderer
0785f83
prmr pushed a commit that referenced this issue Oct 25, 2022
@Laurieprince @prmr
Issue #471 Implement tests for SequenceDiagram builder and renderer
4b400b9
@prmr prmr closed this as completed Oct 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@prmr prmr

Labels
main Issues with the main code base refactoring A proposal to refector or generally improve the code
Projects
None yet
Milestone
Release 3.5
Development

No branches or pull requests
1 participant
@prmr