🔧 Removed babel-preset-es2015 in favour of babel-preset-env

devinekask / generator-devine-boilerplate

⚙ the one and only devine.be boilerplate generator

23 stars 3 forks source link

🔧 Removed babel-preset-es2015 in favour of babel-preset-env #41

Closed thibmaek closed 7 years ago

thibmaek commented 7 years ago

Base for #37 IE >= 11 looked realistic to me ¯_(ツ)_/¯

geoffreydhuyvetters commented 7 years ago

ga hier eerst even over discussieren 'op den bureau' :)

thibmaek commented 7 years ago

Top ;) net nog een foutje gefixt die ik over het hoofd had gezien

geoffreydhuyvetters commented 7 years ago

heb je de production build getest? heb gehoord dat Uglify wat vreemd kan doen

thibmaek commented 7 years ago

Nope, gisteren nogaal traag internet dus niet rechtstreeks kunnen testen jammer genoeg. Let me know als het mocht erroren dan probeer ik es te zien of het fixable is.

geoffreydhuyvetters commented 7 years ago

Ga het eens testen, we gaan voorlopig voor IE10. (kan je het nog aanpassen?) Maar maakt het wat gemakkelijker om in de toekomst te updaten.

thibmaek commented 7 years ago

Sure zal die target snel even veranderen. Zag idd ook de thread op Twitter tussen maintainer van Babel en Kent C. Dodds over Uglify issue :/ Anyway deze PR zou nog steeds perfect mergeable moeten zijn eens ze die fix pushen I guess

geoffreydhuyvetters commented 7 years ago

merged;

met onze target IE10 kom je die Uglify issue niet tegen

thibmaek commented 7 years ago

Top!