-
Notifications
You must be signed in to change notification settings - Fork 89
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
Add support for graphviz #105
Comments
I'll see what I can do about it. It shouldn't be particularly hard to pull off. I'd definitely appreciate a PR, if you can spare the time to try it out, even if it's incomplete. For now, one thing I'm wondering is what the format of the node names can be. Is it allowed to have something like |
Yes something like |
Hey @AndrewRadev,
while having the question in mind: How to decide whether to take the next line into account? Of course they can also be shuffled around. |
I actually wouldn't try to be too clever about this. The way I imagine this particular example to work is in these phases: A -> C;
A -> D;
... goes to ...
A -> C, D;
B -> C;
B -> D;
... goes to ...
B -> C, D;
A -> C, D;
B -> C, D;
... goes to ...
A, B -> C, D So, if you have the exact same text on the left, or on the right, you can join the other sides, otherwise just don't do anything. If they're reordered, the user can easily order them correctly with a couple of |
Hey @AndrewRadev
I really enjoy your plugin and the way you really quickly include new feature requests. Thus, in any circumstance I would want to split anything, I am thinking to request it here. Now I will do so for
graphviz
's dot language for graph specification. The syntax of edges is rather simple:A -> B, C
creates two edges from the node A to both the node B and C.
Now, at some point you want to add a label to one specific edge. They have to be splitted.
The other way round:
could be joined into
A -> B,C
.Please note that multiple elements on the left-hand side are also allowed. This could make splitting things more difficult, but also simplify joining. Thus the following two variants are also equivalent.
A, B -> C, D
I propose to not split or join anything with a square bracket inside, since you can make no assumptions to which edge the attributes belong.
It is not very urgent, but just another nice use-case for your plug-in.
I could also try to work on it in the beginning of april by myself.
The text was updated successfully, but these errors were encountered: