Amazing module... Can this be ported to non-webpack plain nodejs?

[aackerman]

Amazing module... Can this be ported to non-webpack plain nodejs?

[aackerman]

Can you explain how you would expect to use a module in node to discover circular dependencies?

I feel like this could work easily with webpack using the node target, as a part of CI verification.

Without webpack would require overloading node's require() method.

[wlingke]

I was thinking more as a static analysis tool for non-webpack stuff (like
servers).

I think this is also more likely to be run on a one-off basis before a
commit as part of a task than as continuously during dev. The performance
isn't bad but it does slow down my reload when I plug it into my webpack
watcher. Also I don't really think its necessary to be continuously
verifying, it's pretty rare that you'll make circular references.
ᐧ

On Mon, Oct 17, 2016 at 4:42 PM, Aaron Ackerman [email protected]
wrote:

Can you explain how you would expect to use a module in node to discover
circular dependencies?

I feel like this could work easily with webpack using the node target, as
a part of CI verification.

Without webpack would require overloading node's require() method.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#11 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AC__QCvAsU2o2f-p1sYM-9Gd4H6GMmrtks5q1Af8gaJpZM4KZLjt
.

ᐧ

[vitalets]

@wlingke you can use madge for that.
We used it for a long time before and it works fine.

[wlingke]

Ah okay thanks!

…

On Thu, Dec 1, 2016 at 10:12 AM, Vitaliy Potapov ***@***.***> wrote: @wlingke <https://github.com/wlingke> you can use madge <https://github.com/pahen/madge> for that. We used it for a long time before and it works fine. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#11 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AC__QA6pKpk1r25O50A7yyhK_nBgiZ_yks5rDw32gaJpZM4KZLjt> .