Skip to content

Conversation

@ansgarradermacher
Copy link

  • Support of several diagrams: Class, Sequence, composite, and state
  • Several enhancements, e.g. use of qualified names, visibility, static attributes, constructors/destructors

- Support of several diagrams:
  Class, Sequence, composite, and state
- Several enhancements, e.g. use of qualified names, visibility, static attributes,
	constructors/destructors

Signed-off-by: aradermache <ansgar.radermacher@cea.fr>
@travkin79
Copy link
Collaborator

Hi @ansgarradermacher,
Thank you very much for your PR. Could you please also provide a few exemplary UML files for testing purposes or (even better) a few automated tests?

In addition, please add a copyright notice to the header of each new file. There are some new file without such a notice.
BR

@ansgarradermacher
Copy link
Author

Dear @travkin79 ,

Sorry for the late reply (I somehow missed the mail notification). I'll try to add these elements as soon as possible.

BR

@travkin79
Copy link
Collaborator

Hi @ansgarradermacher,
No problem, no hurry. :-)
BR

- Add JUnit test plugin with models and expected plantUML output
- Assure maven compilation
  - Move to JVM 17 and adapted version number to 1.1.33
  - Add plugins to aggregator pom
- Fix error in generalization output
- Add org.eclipse.uml2.uml to target platform (with unspecified version number, is that ok?)
- Add UML2 feature. Problem? Source Header state EPL-v2, feature EPL-v1 licence

Signed-off-by: aradermache <ansgar.radermacher@cea.fr>
@travkin79
Copy link
Collaborator

travkin79 commented Oct 8, 2025

Hi @ansgarradermacher,
Currently, I'm working on merging your PR. But since we moved the development to a new git repo at https://github.com/plantuml/plantuml-eclipse (see plantuml/plantuml-eclipse#2), the merge is more complicated than usually. I will not merge your PR in Hallvard's original git repo, but in the new one. You'll find the latest state in the git branch feature/fix-uml-support.

Add org.eclipse.uml2.uml to target platform (with unspecified version number, is that ok?)

I prefer using explicit versions. I took the version from the target platform definition. I don't know if it would work without the version.

Add UML2 feature. Problem? Source Header state EPL-v2, feature EPL-v1 licence

Originally, the project is licensed under EPL 1.0. I adapted the comments in the Java source code accordingly.

@travkin79
Copy link
Collaborator

Hi @ansgarradermacher,
Thank you for your contribution. It seems, everything works as expected in your PR. I think, I'll merge the changes in the next days and it will be part of the next release. You'll find it here soon:

New git repo: https://github.com/plantuml/plantuml-eclipse
New Eclipse update site: https://plantuml.github.io/plantuml-eclipse/

@travkin79 travkin79 closed this Oct 8, 2025
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