Build configuration #175
Build configuration #175
Conversation
|
Untracked changes made it a little difficult to rebase. But anyway I think this is the minimal configuration we require. I will work on reducing the build size for the |
| @@ -2,4 +2,4 @@ | |||
| node_modules | |||
| *.log | |||
| .DS_Store | |||
| .idea | |||
| build | |||
There was a problem hiding this comment.
why did you get rid of .idea?
There was a problem hiding this comment.
Oh! I'll make the change.
| @@ -0,0 +1,3 @@ | |||
| { | |||
| "presets": ["es2015"] | |||
| } | |||
There was a problem hiding this comment.
new line at end of file pls!
There was a problem hiding this comment.
I think I need to configure prettier settings!
| @@ -0,0 +1,43 @@ | |||
| const webpack = require("webpack"); | |||
| const path = require("path"); | |||
| const BabiliPlugin = require("babili-webpack-plugin"); | |||
There was a problem hiding this comment.
looks like this isn't used?
There was a problem hiding this comment.
Oops! Yup. We rely on uglifyJS
| }), | ||
| new CleanWebpackPlugin([path.resolve(__dirname, "../build")]) // Prebuild (kinda hook) | ||
| ] | ||
| }; |
There was a problem hiding this comment.
new lineeeee (added to the linter issue)
| @@ -0,0 +1,43 @@ | |||
| const webpack = require("webpack"); | |||
There was a problem hiding this comment.
single quotes please! (I'm surprised the linter didn't catch that, maybe it doesn't check for it - will open an issue)
| NODE_ENV: JSON.stringify("production") | ||
| } | ||
| }), | ||
| new CleanWebpackPlugin([path.resolve(__dirname, "../build")]) // Prebuild (kinda hook) |
There was a problem hiding this comment.
Can you update the comments in this file to explain a bit better what things do? kinda hook confuses me
I'm not super familiar with webpack but will look some of this stuff up - but if you could go through this config and explain why you chose this config that'd be good too - thanks!
There was a problem hiding this comment.
Don't worry! I will comment the configuration and will make it easier for you also to make changes afterwards.
| @@ -2,23 +2,34 @@ | |||
| "name": "mathsteps", | |||
| "version": "0.1.6", | |||
| "description": "Step by step math solutions", | |||
| "main": "index.js", | |||
| "main": "./build/mathsteps.min.js", | |||
There was a problem hiding this comment.
if build/ is gitignored, will this not work? or will it always build before looking for main
| "test": "node_modules/.bin/mocha --recursive", | ||
| "setup-hooks": "ln -s ../../scripts/git-hooks/pre-commit.sh .git/hooks/pre-commit" | ||
| "build": "NODE_ENV=production ./node_modules/.bin/webpack --config ./config/webpack.config.js --display-max-modules 0", | ||
| "lint": "node_modules/.bin/eslint ./lib", |
There was a problem hiding this comment.
this removes linting from the test files, which I don't think we should do
|
Thanks :D |
…in42/mathsteps/build-configuration' (PR: google#175)
Added config for
umdbuild.