Use babel-loader instead of jsx-loader about react-starter

[relacsua]

@gaearon You can enable ES6 support with jsx-loader by appending ?harmony and you can see that in the config file.

[gaearon]

I can, but it supports a lot less ES6 than Babel. See http://kangax.github.io/compat-table/es6/

[FoxxMD]

👍

[sokra]

👍

[Rockfordal]

babel works great for me, i excluded node_modules.
(im using reflux, material-ui, socket.io). tnx for a superb starter!

[FoxxMD]

@Rockfordal how did you switch? I've added babel to additional loaders in webpack and concatenated these to the final loaders but it doesn't seem to be working.

I get a module parsing failure on app.jsx in config :/

[sokra]

You need to change these lines:

• https://github.com/webpack/react-starter/blob/master/make-webpack-config.js#L14
• https://github.com/webpack/react-starter/blob/master/make-webpack-config.js#L16-L19

Someone like to do a PR?

[sokra]

As next step we can change the module style to ES6 modules.

[FoxxMD]

@sokra Doesn't using loadersByExtension preclude the possibility of writing your own tests and excludes for any loaders? If I wanted to write my files as normal .js and use babel-loader I wouldn't be able to exclude node_modules, or am I overlooking some functionality in loadersByExtension that allows this?

Also, If I am using additionalLoaders and concat them why would they not be be working? I have commented out the extension loader for jsx and my additionalLoaders look like this:

    var additionalLoaders = [
        { test: /\.js$/, exclude: /node_modules/, loader: "babel-loader" }//,
        { test: /\.jsx$/, exclude: /node_modules/, loader: options.hotComponents ? ["react-hot-loader", "babel-loader"] : "babel-loader" }
    ];

[sokra]

Should also work.
But you can specify exluce and include in loadersByExtension. See https://github.com/webpack/react-starter/blob/master/make-webpack-config.js#L16-L19

[FoxxMD]

That worked! So what is loadersByExtension doing that concatenating additionalLoaders is not?

Also I'll make a PR tomorrow

[pikeas]

@FoxxMD #42

[sokra]

#60