search

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

Support outputting raw ".dot" files with the DotCommand #54

Closed Pierstoval closed 5 years ago

Pierstoval commented 5 years ago

I wanted to customize the output of Graphviz, and it becomes possible with this command

coveralls commented 5 years ago

Coverage Status

Coverage remained the same at 100.0% when pulling f5eb49ff596e135b9260b0152ebffdbc708f1019 on Pierstoval:dot into 4b4f8b34311a13e2f37fab53245d269cab746190 on mihaeu:master.

mihaeu commented 5 years ago

Thanks for the PR 👍

You know about --keep, but would like to have the option of not having any waste output, right?

Sort of like a shortcut for --keep --output=/dev/null, right?

Pierstoval commented 5 years ago

Yep, that's what I did in the first place to do it myself, but it is a "hack" :)

This PR introduces a native way to do it, which I think is much more intuitive 😄

mihaeu commented 5 years ago

Alright, you got me convinced 😉

On Wed, Aug 7, 2019, 02:27 Alex Rock notifications@github.com wrote:

Yep, that's what I did in the first place to do it myself, but it is a "hack" :)

This PR introduces a native way to do it, which I think is much more intuitive 😄

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/mihaeu/dephpend/pull/54?email_source=notifications&email_token=AAQRO7JIJHHDV7E36M4DGYTQDJ2OZA5CNFSM4IJXIXQKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3XPD7A#issuecomment-518975996, or mute the thread https://github.com/notifications/unsubscribe-auth/AAQRO7O3CJ3SZFEFUWF2CX3QDJ2OZANCNFSM4IJXIXQA .

Pierstoval commented 5 years ago

🎉 😊