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

Provide some helper for merged bundle #119

Open
tadatuta opened this issue Apr 4, 2015 · 2 comments
Open

Provide some helper for merged bundle #119

tadatuta opened this issue Apr 4, 2015 · 2 comments

Comments

@tadatuta
Copy link
Member

tadatuta commented Apr 4, 2015

In most of the cases merged bundle differ from other ones just in the step of getting declaration.
So it should be possible to write a module which would get all the techs from nodeConfig (or whatever?) and replace that step.

As a result the API should look like this:

config.nodes('*.bundles/merged', function (nodeConfig) {
    nodeConfig.addTechs (require('enb-bem-merged-bundle')(nodeConfig));
    nodeConfig.addTargets(['?.css', '?.js']);
});
@qfox
Copy link
Contributor

qfox commented Apr 4, 2015

In common, please don't use conditionals for configurations. Complexity shouldn't grow here...

@tadatuta
Copy link
Member Author

tadatuta commented Apr 4, 2015

for WIP please refer to https://github.com/bem/project-stub/compare/merged?expand=1

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

No branches or pull requests

3 participants