jedireza/aqua

cheap-module-source-map for production ?

adantoscano opened this issue · 4 comments

Hi,
I found in webpack that it declares mapping with "source-map" option in dev mode and cheap-module-source-map for production. Shoudn't be the opposite?

    let devtool = 'source-map';

    if (process.env.NODE_ENV === 'production') {
        plugins.push(new Webpack.optimize.UglifyJsPlugin({
            compress: {
                warnings: false
            }
        }));

        devtool = 'cheap-module-source-map';
    }

I don't think so. I forget why cheap-module-source-map was chosen for production. Do you think it should be different in production?

You can see here https://webpack.js.org/configuration/devtool/ the tab that cheap is for a fast rebuild but not for production and the oposite with source-map.

image

Want to open a PR with the fix you're suggesting?

I've decided to archive this project. Thank you for your interest and contributions.
https://github.com/jedireza/aqua#no-longer-maintained