Update browserify - Githubissues

MergeMyPullRequest / order-splitter

STABLE: https://mergemypullrequest.github.io/order-splitter/ BETA: https://jonsmithers.github.io/order-splitter/

MIT License

2 stars 0 forks source link

Update browserify #32

Open jonsmithers opened 7 years ago

jonsmithers commented 7 years ago

The following syntax causes browserify to explode:

Object.entries({a:1,b:2}).forEach(([key, value]) => { });

We're using gulp-browserify which is no longer maintained and several years out of date. It seems to me this syntax bug is long gone in current versions.

Workarounds:

use function instead of =>
assign function to a variable first

jonsmithers commented 7 years ago

I'm also okay with removing browserify...

jonsmithers commented 6 years ago

I'm also okay with switching to webpack...

jonsmithers commented 6 years ago

:astonished: Rob Dodson wrote an article about using Polymer with webpack!

jonsmithers commented 6 years ago

Safest solution is probably to model our build after one of the example browserify recipes.