search

enb / enb-bem-techs

ENB package to build BEM-projects
Other
22 stars 22 forks source link

use node-eval #230

Closed blond closed 7 years ago

blond commented 8 years ago

We can use node-eval module instead of enb-async-require and enb-require-or-eval.

It is possible to decrease memory consumption: require cache is not used.

After the changes we can not substitute techs in tests: https://github.com/enb/enb-bem-techs/blob/master/test/utils/techs.js.

skad0 commented 8 years ago

Agreed with @Yeti-or and @blond that we should integrate node-eval into enb-require-or-eval.

blond commented 7 years ago

Fixed in https://github.com/enb/enb-bem-techs/pull/243