Skip to content

Update PlantUML Maven Plugin and drop requirement for dot#700

Closed
mthmulders wants to merge 3 commits intoapache:masterfrom
infosupport:fix/630-newer-plantuml-plugin-without-dot
Closed

Update PlantUML Maven Plugin and drop requirement for dot#700
mthmulders wants to merge 3 commits intoapache:masterfrom
infosupport:fix/630-newer-plantuml-plugin-without-dot

Conversation

@mthmulders
Copy link
Contributor

Following this checklist to help us incorporate your
contribution quickly and easily:

  • Your pull request should address just one issue, without pulling in other changes.
  • Each commit in the pull request should have a meaningful subject line and body.
    Note that commits might be squashed by a maintainer on merge.
  • Run mvn site and examine output in target/site directory.
    Site will also be built on your pull request automatically and attached to GitHub Action result.

If your pull request is about ~20 lines of code you don't need to sign an
Individual Contributor License Agreement if you are unsure
please ask on the developers list.

To make clear that you license your contribution under
the Apache License Version 2.0, January 2004
you have to acknowledge this by using the following check-box.

Closes #630.


legend
{{
!pragma layout smetana
Copy link
Member

Choose a reason for hiding this comment

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

any reason this is not affected by the global pragma?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No clue. Trial and error showed me that the nested diagram needs this separately. Also happened when I had the diagram open in VS Code, so it seems to me it's not a Maven plugin issue.

@mthmulders
Copy link
Contributor Author

Closed in 10d086a.

@mthmulders mthmulders closed this Feb 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

plantuml plugin requires dot utiliy

5 participants