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

intersect-deps and intersect-bemdecl techs #190

Closed
wants to merge 43 commits into from
Closed

intersect-deps and intersect-bemdecl techs #190

wants to merge 43 commits into from

Conversation

JiLiZART
Copy link

Use case
Build a common bemdecl or deps file that consists only from blocks used in all bundles.
After i can substract this bemdecl file from other bundles and generate common.css and common.js + bundle.css and bundle.js file.

TODO

  • more bemdecl tech tests
  • more deps tech tests

@blond blond added the review label Jan 15, 2016
@JiLiZART
Copy link
Author

/cc @blond
Я от тебя не отстану :)

@blond
Copy link
Member

blond commented Jan 15, 2016

@JiLiZART, задумка зачёт! ;)

На код внимательно посмотрю на днях.


this._target = this.getOption('bemdeclTarget');
if (this._target) {
logger.logOptionIsDeprecated(this.node.unmaskTargetName(this._target), 'enb-bem', this.getName(),
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

В новой технологии не нужно поддерживать устаревшие опции.

@blond
Copy link
Member

blond commented Jan 28, 2016

@JiLiZART, есть предложение сделать базовую технологию для объединения, вычитания и пересечения.

sources: [a.deps.js, b, c]
sources: [a, b, c]

Для объединения и пересечения всё очень похоже, для вычитания немного отличается. Что-то можно вынести в хелпер. Чтобы в итоге технологии отличались только работой с данными (сейчас это пару строк).

@blond blond added ready and removed review labels Jan 28, 2016
@JiLiZART
Copy link
Author

@blond если я правильно понял, сделать базовую технологию, в которой можно будет выбирать что конкретно делаем, вычитаем, пересекаем или объединяем?
Или просто объединить похожий код в хелпер функции?

@blond
Copy link
Member

blond commented Jan 28, 2016

если я правильно понял, сделать базовую технологию, в которой можно будет выбирать что конкретно делаем, вычитаем, пересекаем или объединяем?

Скорее всего так не получится.

Или просто объединить похожий код в хелпер функции?

А так точно получится :)

@JiLiZART
Copy link
Author

ok

@blond
Copy link
Member

blond commented Mar 17, 2016

@JiLiZART, какие успехи? )

@JiLiZART
Copy link
Author

На выходных буду фиксить

@coveralls
Copy link

Coverage Status

Coverage increased (+0.01%) to 95.161% when pulling cb2e11a on JiLiZART:intersectTechs into a877bed on enb:master.

@JiLiZART
Copy link
Author

@blond В общем, долгие у меня выходные видимо выдались.

@qfox
Copy link
Contributor

qfox commented Aug 22, 2016

Ребят, а почему вы не хотите использовать bem-decl с его интерсектами?

@blond
Copy link
Member

blond commented Aug 29, 2016

@zxqfox теперь, конечно, надо хотеть.

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

Successfully merging this pull request may close these issues.

None yet

6 participants