enb / enb-bem-i18n

BEM internationalization for ENB
Other
8 stars 7 forks source link

Implement merge for i18n-js tech #38

Closed tormozz48 closed 9 years ago

tormozz48 commented 9 years ago

resolved #33 @blond This is continuation of pr #37

coveralls commented 9 years ago

Coverage Status

Coverage increased (+2.69%) to 67.39% when pulling a09913e698173ca4713ef4fb47976a15a07cd640 on issue-33 into afcd573ef219dba512effa3d271e4ca7e964082d on 1.x.

tormozz48 commented 9 years ago

На всякий случай продублирую вопросы которые остались:

  1. "Для старого i18n потеряли возможность собрать несколько языковых кейсетов в одном файле, чтобы потом просто переключать BEM.I18N.lang('current-lang');"
  2. "Может fixtures будем выкачивать через bower?"
blond commented 9 years ago

"Может fixtures будем выкачивать через bower?" - тянуть всю библиотеку?

Даже 2 :)

"Для старого i18n потеряли возможность собрать несколько языковых кейсетов в одном файле, чтобы потом просто переключать BEM.I18N.lang('current-lang');"

Это критично? Как этого избежать?

Думаю, что этим пользовались. Если это оторвать, то любители считать байты расстроятся от того, что придётся держать в коде ядро несколько раз.

Можно подумать про опцию langs + defaultLang вместо lang. Но не ясно, стоит ли давать такую функциональность для нового i18n.

blond commented 9 years ago

Нужно отребейзить.

coveralls commented 9 years ago

Coverage Status

Coverage increased (+2.69%) to 67.39% when pulling 197026e8fb13bf9962937963b362e44e6907770f on issue-33 into 9c44ecf1405a99ecb032150ff06761b0fe5a761a on 1.x.

tormozz48 commented 9 years ago

Создал связанную задачу на будущее https://github.com/enb-bem/enb-bem-i18n/issues/42

blond commented 9 years ago

В AppVeyor bower i не запускается на претесты, т.к. там запуск через npm run unit.

blond commented 9 years ago

Надо бы разделить тесты на несколько файлов, если и есть что-то общее, пусть лучше скопипаститься, зато будет чётко видно, что гарантируется с этим ядром.

blond commented 9 years ago

В остальном :ok:

blond commented 9 years ago

:+1: