pumascript / puma

Meta-programming framework for JavaScript based on LayerD concepts
MIT License
30 stars 28 forks source link

Delete parameter "Passed!" and Review Skiped #63 #103

Closed Agu-GC closed 7 years ago

Agu-GC commented 7 years ago
Undre4m commented 7 years ago

@Agu-GC Gracias por el aporte! 👍 Te está fallando el build xq cuando reemplazaste las comillas dobles, las simples que estaba englobadas en el string hacen que se rompa.

Por ej: 'function strict(){\'use strict\';function nested(){return \' function\';}return \'strict mode into\'+nested();} strict();' hace que el string termine en el {\ y use strict que fuera del string y salta el error. Yo dsps lo veo en más detenimiento, pero dejaría las comillas dobles como estaban...

También fijate que en puma/test/suites/ecma-executable-code-and-execution-contexts-test.js y test/suites/ecma-type-conversion-and-testing.js tenés problemas de indentación. Además hay partes que parecería que fueron reemplazadas al mergear estos archivos, como las directivas @file. Seguro de alguna resolución de conflictos...

Undre4m commented 7 years ago

En resumen, los dos últimos commits están bien a mi criterio. Esperaría a ver que dicen los chicos pero creo que lo mejor es pararte en el ultimo commit antes del merge, hacer un revert del replace double quotes to simple quotes y volver a mergear con master, donde seguro te tira conflictos en archivos que estaban mal indentados.

emravera commented 7 years ago

Vamos a cerrar este PR. Ya hable con Agus y vimos los cambios va a subir uno nuevo con las correcciones.