apache / cordova-coho

Apache Cordova coho
Apache License 2.0
33 stars 62 forks source link

Add *draft* tools dependency graph by @raphinesse #186

Closed brodycj closed 6 years ago

brodycj commented 6 years ago

Platforms affected

All

What does this PR do?

Add draft tools dependency graph from https://github.com/apache/cordova-discuss/issues/101#issuecomment-402857794 by @raphinesse:

cordova-tooling

Thanks for guidance: https://stackoverflow.com/questions/10189356/how-to-add-screenshot-to-readmes-in-github-repository#10189374

To be merged by squash if there are no objections within the next 1-2 days

What testing has been done on this change?

Visual inspection

Checklist

raphinesse commented 6 years ago

Thanks for finding a place to put this. registry-mapper has been removed since I created this. I'll try to find some time to update it.

brodycj commented 6 years ago

registry-mapper has been removed since I created this

in master branch only - added a quick note with this info

I'll try to find some time to update it.

Considering that this is not yet the case in existing release I would not rush it. Of course the quick solution would be to use "eraser" tool.

Some minor feedback in case you decide to work on the graph again: it is not so clear to me from first glance what order I should publish things in to ensure they are all published with updated dependencies. Reading through the graph I think updates should be published in the following rough order (may not be exact):

I think a top-down/bottom-up graph would be more clear. A couple examples from https://www.ndepend.com/docs/visual-studio-dependency-graph:

raphinesse commented 6 years ago

An online dependency graph editor with capabilities for generating a reverse topological order for the graph would be nice indeed. Unfortunately, I could not find such a tool yet.

Edit: there's http://www.webgraphviz.com but I'd actually like to have something a bit prettier

brodycj commented 6 years ago

Merging (better than nothing, to say the least)

raphinesse commented 6 years ago

So I finally found an editor that almost did what I wanted and could be easily modified. Here it is.

Gives you an edit and an embed URL. Cordova Tooling dependency graph

Y-Coordinates give a topological ordering. So when updating things bottom to top, you should be fine.