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.

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

Feat: Changing control flow naming to tilde #24

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

01Parzival10
Copy link
Contributor

Changing control flow naming to ~ from empty.
Adds additional ~ for multiple incoming control flows to ensure unique names.

@01Parzival10 01Parzival10 added the enhancement New feature or request label Nov 19, 2024
@01Parzival10
Copy link
Contributor Author

This fixes the converter side of DataFlowAnalysis/WebEditor#51.
The Editor still needs to be adjusted.

However, I suggest we merge this before the Editor is adjusted. While both the old and new syntax dont get accepted by the Editor, the new at least allows From-PCM-Created-Jsons to be converted into DFD's again

@sebinside
Copy link
Member

Needs adjustments from @Kr0nox in the web editor

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants