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

Webpack 4 + babel 7 final #10

Closed
wants to merge 17 commits into from

Conversation

duGuillaume
Copy link
Contributor

Babel 7 is out and this PR (#2) look dead so, here a new PR with a clean package.json and webpack.config

i use webpack 4 and babel 7.
packages have been updated.
i created an img folder into assets and removed thems from css folder.
fonts from css folder have been removed.

instead of 'npm run watch' we use 'npm start'
now .map files are create with watch command to help developpement
A new npm command have been created to help debug (can be improved)

@kpodemski
Copy link

kpodemski commented Sep 4, 2018

Hey,

curious, what is your time for:

  1. build
  2. start
  3. start, 2nd compilation

?

@duGuillaume
Copy link
Contributor Author

duGuillaume commented Sep 4, 2018

sry i'm using windows wsl, I don't think it's the best config to test timers

@labsin
Copy link

labsin commented Nov 4, 2018

Works here. Have no metrics to test this, but first build is 15s instead of 30s (on an old laptop on arch)

Should _dev/package-lock.json be removed from the PR? It's not in the repo.

@duGuillaume
Copy link
Contributor Author

i'll make an update tonight, some packages need to be change or updated.
i think i'll remove --mode development for 'npm start' command. devtool and source-map shouldn't be here.
I need to remove uglifyjs package to use terser here from the uglifyjs-webpack-plugin changelog.

Switch back to uglify-js (uglify-es is abandoned, if you need uglify ES6 code please use terser-webpack-plugin).

@labsin I think package-lock.json should be in the repo here the doc about it: https://docs.npmjs.com/files/package-lock.json. but it's a good question, what to do with sourcemap ?

@duGuillaume
Copy link
Contributor Author

duGuillaume commented Dec 3, 2018

i removed .map and update packages to use terser instead uglifyjs

@SharakPL
Copy link

what's up with the conflicts? will it be merged anytime soon?

@duGuillaume
Copy link
Contributor Author

I ve updated packages and I'm working on the conflicts i'll make a pull request soon

@duGuillaume
Copy link
Contributor Author

conflicts merged and packages updated

@jdziewonski
Copy link

jdziewonski commented Sep 2, 2019

What's up with this PR ?
I have my own updated webpack version and i'm considering creating new PR with my own solution, but it seems that this PR needs only simple merge, npm update and sources compilation.

Do @prestarocket consider that update as proper solution ? There is gulpfile.js on develop branch, and it's there only to compile sass files, but updated webpack handle it well.
If performance is to bad, maybe consider changing sass-loader to fast-sass-loader as mentioned in here

@prestarocket
Copy link
Collaborator

What's up with this PR ?
I have my own updated webpack version and i'm considering creating new PR with my own solution, but it seems that this PR needs only simple merge, npm update and sources compilation.

Do @prestarocket consider that update as proper solution ? There is gulpfile.js on develop branch, and it's there only to compile sass files, but updated webpack handle it well.
If performance is to bad, maybe consider changing sass-loader to fast-sass-loader as mentioned in here

could ypu make a new PR on dev.b branch, i will test it ? i m really not an expert with webpack...

@prestarocket
Copy link
Collaborator

finally !! 2238019

@prestarocket
Copy link
Collaborator

thank you, your work has helped me a lot!

@dangelion
Copy link

Is this currently merged? I still see webpack 2 here

@prestarocket
Copy link
Collaborator

prestarocket commented Mar 27, 2020 via email

@dangelion
Copy link

@prestarocket is it still in development? For when you think it will be ready for production?

@dangelion
Copy link

@prestarocket can I use dev branch for a production project? Is it quiet stable?

@prestarocket
Copy link
Collaborator

yes

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

Successfully merging this pull request may close these issues.

7 participants