search

andrewplummer / Sugar

A Javascript library for working with native objects.
https://sugarjs.com/
MIT License
4.53k stars 306 forks source link

Don't know what happened from 1.4.1 to 1.5.0 #563

Closed XadillaX closed 7 years ago

XadillaX commented 7 years ago

Old version is 1.4.1 and everything is OK.

After I installing 1.5.0, errors occurred but I don't know why.

RangeError: Maximum call stack size exceeded at getOwnPropertyNames (native) at iterateOverPropertyNames 
(/Users/*/Workspace/sealionx/feime/node_modules/.npminstall/sugar/1.5.0/sugar/sugar-full.js:6893:7) at iterateOverProperties 
(/Users/*/Workspace/sealionx/feime/node_modules/.npminstall/sugar/1.5.0/sugar/sugar-full.js:6886:9) at mergeObject 
(/Users/*/Workspace/sealionx/feime/node_modules/.npminstall/sugar/1.5.0/sugar/sugar-full.js:6909:9) at 
/Users/*/Workspace/sealionx/feime/node_modules/.npminstall/sugar/1.5.0/sugar/sugar-full.js:6932:15 at Array.forEach (native) at iterateOverPropertyNames 
(/Users/*/Workspace/sealionx/feime/node_modules/.npminstall/sugar/1.5.0/sugar/sugar-full.js:6893:32) at iterateOverProperties 
(/Users/*/Workspace/sealionx/feime/node_modules/.npminstall/sugar/1.5.0/sugar/sugar-full.js:6886:9) at mergeObject 
(/Users/*/Workspace/sealionx/feime/node_modules/.npminstall/sugar/1.5.0/sugar/sugar-full.js:6909:9) at 
/Users/*/Workspace/sealionx/feime/node_modules/.npminstall/sugar/1.5.0/sugar/sugar-full.js:6932:15 at Array.forEach (native) at iterateOverPropertyNames 
(/Users/*/Workspace/sealionx/feime/node_modules/.npminstall/sugar/1.5.0/sugar/sugar-full.js:6893:32) at iterateOverProperties 
(/Users/*/Workspace/sealionx/feime/node_modules/.npminstall/sugar/1.5.0/sugar/sugar-full.js:6886:9) at mergeObject 
(/Users/*/Workspace/sealionx/feime/node_modules/.npminstall/sugar/1.5.0/sugar/sugar-full.js:6909:9) at 
/Users/*/Workspace/sealionx/feime/node_modules/.npminstall/sugar/1.5.0/sugar/sugar-full.js:6932:15 at Array.forEach (native)
andrewplummer commented 7 years ago

I'm not seeing anything on my end... can you post your node version? Did you do a simple npm install?

andrewplummer commented 7 years ago

Going to close this one for now. Let me know if it's still an issue!