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

PlantUmlJob extension with system and allocation. #55

Merged
merged 3 commits into from
Jan 25, 2024
Merged

PlantUmlJob extension with system and allocation. #55

merged 3 commits into from
Jan 25, 2024

Conversation

dr6817
Copy link
Member

@dr6817 dr6817 commented Jan 24, 2024

No description provided.

@dr6817 dr6817 requested a review from FloBoJa January 24, 2024 16:38
This will not be needed until the Palladio-Addons-PlantUML supports
ResourceEnvironments, which might still be far away. Better keep the
code base clean until then.
Copy link
Collaborator

@FloBoJa FloBoJa left a comment

Choose a reason for hiding this comment

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

I removed the unused argument since that resource environment change might take a while still. We can add it back once we are there. I also made sure everything was formatted correctly.

The diagrams look awful right now (or do not work at all!), but that is something to take care of in Palladio-Addons-PlantUML, not in this PR.

@dr6817 dr6817 merged commit 90e791a into main Jan 25, 2024
9 checks passed
@dr6817 dr6817 deleted the uml branch January 25, 2024 08: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.

2 participants