fix: circular dependencies result in endless loop + out of memory crash about live-plugin-manager HOT 3 OPEN

For me we can leave the issue open for the future, so I can monitor it.

from live-plugin-manager.

Circular requires should be supported, but not sure about circular dependencies.
From high level point of view it sounds strange... but probably in someway we can support it.

AS a workaround you can probably install es6-iterator and es5-ext manually and then mark them as a static dependencies? (see staticDependencies options)

from live-plugin-manager.

@davideicardi

Circular requires should be supported, but not sure about circular dependencies.

Fair point IMO. The only reason why I'd still consider dealing with it due to wanting to be consistent with the other package managers (both npm and yarn are able to install these dependencies). Of course I also understand 100% that if being consistent in all fronts like that is not part of the vision of the project.

AS a workaround you can probably install es6-iterator and es5-ext manually and then mark them as a static dependencies? (see staticDependencies options)

I had to go another route so I'm unable to test it anymore unfortunately. (I'm using npm's --prefix parameter to install plugins "somewhere else on the file-system")
Based on the above I'm also okay with just closing this of course, if no one else had this issue and I've moved on then I don't want to take up your time!

from live-plugin-manager.