landscaper: Add a cycle detection script to test process

canjs / can-connect

Model layer utilities for every JavaScript framework! Assemble real-time, high performance, restful data connections.

https://canjs.com/doc/can-connect.html

MIT License

29 stars 16 forks source link

landscaper: Add a cycle detection script to test process #357

Open bmomberger-bitovi opened 6 years ago

bmomberger-bitovi commented 6 years ago

This pull request was created with Landscaper. The following code mods were used to create this PR:

detect-cycles-codemod

Please review this PR carefully as Landscaper does not guarantee any code mod's quality.

bmomberger-bitovi commented 6 years ago

can-connect has a cyclic dependency on can-fixture. Can-fixture uses the can-connect memory cache behavior, while can-connect uses fixtures for all of its tests (which is not really necessary -- a fake ajax function would work just as well). I don't have the time to allocate to fixing it at this time, so this will remain in limbo unless someone want to take on breaking the cycle.