search

enb / enb-bem-i18n

BEM internationalization for ENB
Other
8 stars 7 forks source link

#66 Add exports option to i18n technology #67

Closed tormozz48 closed 9 years ago

tormozz48 commented 9 years ago

Resolved #66

@blond @arikon

Please review it

blond commented 9 years ago

А зачем такие сложности с дополнительными функциями?

Почему не что-то в таком духе exports.globals ? 'code' : ''?

tormozz48 commented 9 years ago

А зачем такие сложности с дополнительными функциями?

Ну мы же не ищем легких путей :smile: А если серьезно, то как по мне такой код более структурирован и понятен чем то что ты предлагаешь.

P.S. Конечно, если ты будешь настаивать, то я перепишу его.

blond commented 9 years ago

По мне такой код сложнее читать. Когда все строчки рядом читается почти как обычный код. Когда функции, то нужно думать какой код получится.

tormozz48 commented 9 years ago

Убрал функции.

tormozz48 commented 9 years ago

Исправил и объединил коммиты