search

enb / enb-bem-i18n

BEM internationalization for ENB
Other
8 stars 7 forks source link

Actualize jsdoc #56

Closed tormozz48 closed 9 years ago

tormozz48 commented 9 years ago

Resolved #50

@tadatuta @tavriaforever @blond Please review it

Attention: There are broken tests due to last changes in enb-xjst package. I propose to fix them in separate issue.

arikon commented 9 years ago

@tormozz48 Пиши нормальные описания к pr, пожалуйста. Issue 50 — ни о чём не говорит.

tormozz48 commented 9 years ago

@tadatuta @innabelaya

Please review this pr

tormozz48 commented 9 years ago

@innabelaya ты усе?

innabelaya commented 9 years ago

@tormozz48 кажись, да :)

tormozz48 commented 9 years ago

@blond

Поправил все замечания от Инны. Объединил коммиты.

P.S. Забавно то, что я копипастил jsdoc из enb-bh, enb-bemxjst, enb-xjst и Инна нашла кучу замечаний в строках jsdoc, который у нас присутствует во всех вышеперечисленных пакетах.

tadatuta commented 9 years ago

Забавно то, что я копипастил jsdoc из enb-bh, enb-bemxjst, enb-xjst и Инна нашла кучу замечаний в строках jsdoc, который у нас присутствует во всех вышеперечисленных пакетах.

Кто-нибудь портирует такие правки в вышеперечисленные пакеты?

blond commented 9 years ago

Кто-нибудь портирует такие правки в вышеперечисленные пакеты?

@tormozz48 Можешь этим заняться?

tormozz48 commented 9 years ago

@tormozz48 Можешь этим заняться?

да, но не в срочном порядке

tormozz48 commented 9 years ago

Предлагаю все-таки влить этот pr и закрыть задачу

innabelaya commented 9 years ago

@tormozz48 я могу влить эти правки. jsdoc для enb-bh я не смотрела. Но спасибо за твою наблюдательность)

blond commented 9 years ago

@innabelaya было бы круто! Можешь за одно посмотреть на enb-bh?

innabelaya commented 9 years ago

@blond Да, конечно.