search

auchenberg / dependo

Visualize your CommonJS or AMD module dependencies in a force directed graph report.
MIT License
484 stars 48 forks source link

Madge dependencies #15

Closed malys closed 9 years ago

malys commented 9 years ago

Madge has solved dependency issue. https://github.com/pahen/madge/commit/f7e0ff81c7d99797a08d52941d0d568aae983373

Coud you upgrade Madge dependencies to inherit this fix?

Thank you.

auchenberg commented 9 years ago

The latest published version of madge to NPM is 0.3.5.

Which version is needed for this "fix"? Please describe the problem.

malys commented 9 years ago

I' using shrinkwrap to manage dependencies Madge 0.3.1 includes: amdetective coffee-script colors commondir detective graphviz grunt grunt-cli grunt-mocha-test grunt-release resolve should uglify-js walkdir shrinkwrap detects errors with dev dependencies. It's normal because when you export a node module to npm registry you haven' to include dev modules.

Madge 0.3.5 includes: amdetective coffee-script colors commander commondir detective graphviz resolve uglify-js walkdir Pahen excludes dev dependencies. That is the good way to manage dependencies.

From Madge0.3.4, the issue was fixed.

Thank you

auchenberg commented 9 years ago

Madge is hereby updated to 0.3.5.

malys commented 9 years ago

Thank you.