search

eugeneware / debowerify

A browserify transform to enable the easy use of bower components in browserify client javascript projects. This can be used in conjunction with deamdify to require AMD components from bower as well.
493 stars 51 forks source link

Make debowerify work with more than 14 entries #71

Closed mkoryak closed 9 years ago

mkoryak commented 9 years ago

after 14 entries or so, next entries will overwrite code and cause syntax errors, see #67

this fixes that.

maxnordlund commented 9 years ago

I'm a afraid to merge anymore, @eugeneware could you take a look?

mkoryak commented 9 years ago

shouldnt be too afraid, master is broken right now

mkoryak commented 9 years ago

i needed to do the nested arrays thing because you cant safely splice into that array without also moving the computed node boundaries - which we cant do. its easier to add the extra paths as nested array and flatten them later.

mkoryak commented 9 years ago

I am testing this on a big project and i found a bug :)