mihaeu / dephpend

Detect flaws in your architecture, before they drag you down into the depths of dependency hell ...
https://dephpend.com
MIT License
530 stars 26 forks source link

Workaround if you can't install PlantUML #38

Closed nilsabegg closed 6 years ago

nilsabegg commented 6 years ago

It's not a perfect solution for the problem but it works.

coveralls commented 6 years ago

Coverage Status

Coverage remained the same at 100.0% when pulling 115412f0bc115b53e6b3501bb1e946d22d4f7b98 on nilsabegg:develop into 61973bfc09aed042f8a541956550a5151a33a7be on mihaeu:develop.

mihaeu commented 6 years ago

Hi @nilsabegg thanks for your help :+1:

I'll merge this as is, but regarding the actual solution:

1) I was thinking of implementing mermaid.js support which renders PlantUML syntax using html + js. The result would not be a .svg or .png however but instead a html file. 2) How do you feel about adding https://hub.docker.com/r/think/plantuml/ to your text? 3) Another idea would be to add an option like --plantuml-cmd="docker run --rm -i think/plantuml" which would solve that problem. 4) Check if java is installed, download the .jar file and use that as a fallback.

What do you think?