search

goto-bus-stop / split-require

commonjs-first bundle splitting, for browserify
https://npmjs.com/package/split-require
MIT License
82 stars 8 forks source link

Ensure dependencies of mainRows are added in deterministic order #23

Closed goto-bus-stop closed 5 years ago

goto-bus-stop commented 5 years ago

The row.deps object contains dependency IDs in the order that they were loaded in by module-deps. That's done concurrently so the order isn't always the same. This patch sorts the list of dependencies before adding them to the bundle.

casr commented 5 years ago

Works for me too :)

danielcardoso5 commented 5 years ago

I can confirm this works! Tested on my main project and works great, thanks :)

goto-bus-stop commented 5 years ago

:package: 3.1.2