Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Producción es5 #2

Open
andressul opened this issue Jun 6, 2017 · 3 comments
Open

Producción es5 #2

andressul opened this issue Jun 6, 2017 · 3 comments
Labels

Comments

@andressul
Copy link

La librería producción no debería estar en es5?

Tengo problemas al tratar compilar con webpack en producción

ERROR in app.js from UglifyJs
SyntaxError: Unexpected token: name (options) [./~/@platzi/markdown/index.js:17,0]

@sergiodxa
Copy link
Contributor

sergiodxa commented Jun 6, 2017

Debería, así la usamos en PlatziDev/pulse, aunque usamos Babili para minificar en vez de UglifyJS 🤔 . Voy a verificar que ocurre.

@sergiodxa sergiodxa self-assigned this Jun 7, 2017
@sergiodxa sergiodxa added the bug label Jun 7, 2017
@sergiodxa
Copy link
Contributor

@andreider04 ya confirmé que esto ocurre, tengo que ver si es algo de pulse-editor o de una dependencia de este 🤔 mientras te recomiendo usas Babili para minificar en vez de UglifyJS, Babili soporta sintaxis de ES2015.

@sergiodxa sergiodxa added this to the 1.0.1 milestone Jun 7, 2017
@sergiodxa sergiodxa removed this from the 1.2.0 milestone Jun 25, 2017
@HanselMRojas
Copy link

HanselMRojas commented Sep 25, 2017

Hola espero esto sea de ayuda. Yo intente implementar Pulse editor en mi proyecto. Intente hacer lo de babili. Pero no pude hacerlo funcionar.

Teniendo en cuenta que el problema principal es que cuando se usa UglifyJS no soporta es2015. Lo que puedes hacer es reemplazar el modulo uglifyjs por uglifyjs-es. Y ya existe un plugin para webpack que funciona bastante bien. Esto funciono para mi:

1 Instalar el plugin

# Via Yarn 
[path/mi/proyecto] $ yarn add -D uglify-es-webpack-plugin

# Via npm
[path/mi/proyecto] $ npm install -D uglify-es-webpack-plugin

2 Cambiar plugin en la configuracion webpack

const webpack = require('webpack')
const ExtractTextPlugin = require('extract-text-webpack-plugin')
const UglifyEsPlugin = require('uglify-es-webpack-plugin') // Importar el plugin

/** 
 * Reemplazar webpack.optimize.UglifyJsPlugin por => UglifyJsPlugin()
 * ya que estamos importanto el plugin uglify-es-webpack-plugin
 */

/* Reemplazar esto */
{
    /* ... codigo de configuracion  */
    "plugins": [
        new webpack.optimize.UglifyJsPlugin({
          compress:{
            warnings: false
          },
          mangle: {
            except: ['$super', '$', 'exports', 'require'],
          },
       })
    ]
}

/*  Por esto */
{
  /* ... codigo de configuracion  */
    "plugins": [
        new UglifyEsPlugin()
    ]
}

Esto lo solucionó para mi espero te sirva.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants