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.

Sign up for GitHub

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

Jump to bottom

Add webpack config for bempack #277

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Add webpack config for bempack #277

wants to merge 2 commits into from

Conversation

Yeti-or
Copy link
Member

@Yeti-or Yeti-or commented Sep 14, 2017 •
edited
Loading

TODO:

  • add webpack build process to README.md
  • move bh-render-plugin to separate package
  • add support for bem-xjst
  • depsByTech
  • webpack 2
  • webpack 3

good to have:

  • custom bem-naming
  • custom bem-fs-scheme
  • support bem-config

@Yeti-or Yeti-or requested a review from tadatuta September 15, 2017 00:08
@tadatuta tadatuta changed the title Add webpack config for bempack. Add webpack config for bempack Sep 15, 2017
@tadatuta
Copy link
Member

Looks fine for WIP but I don't want to merge it in current state. Let's keep it in brunch for some time?

@Yeti-or
Copy link
Member Author

Yeti-or commented Sep 26, 2017

webpack:

du -h desktop.bundles/index/index.min.js
396K	desktop.bundles/index/index.min.js

enb:

du -h desktop.bundles/index/index.min.js
248K	desktop.bundles/index/index.min.js

but we could do it: 💪

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tadatuta tadatuta Awaiting requested review from tadatuta

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Yeti-or @tadatuta