javascript_ru

26 October 2017 18:06

Не-не-не, не слушайте его!

javascript_ru

26 October 2017 18:06

один на телегу другой на хмпп, 3 на шину

javascript_ru

26 October 2017 18:06

Если есть иное экспертное мнение — прошу предъявить. Буду склонен согласиться :)

javascript_ru

26 October 2017 18:06

Но чот рационализировать затрудняюсь

javascript_ru

26 October 2017 18:05

С докерфайлом никогда не работал, сделал просто по аналогии. Я правильно понял, файл должен быть таким? https://pastebin.com/PTfWPmHX

javascript_ru

26 October 2017 18:03

А вы разобрались там с кейсом в файлах?

javascript_ru

26 October 2017 18:02

так и пусть будет, хрен редьки слаще же

javascript_ru

26 October 2017 18:02

Ну вот теперь будет Flow, и нам никуда не деться.

javascript_ru

26 October 2017 18:01

Инструкции по сборке и пр. обычно дают с долларами и помечают как console

javascript_ru

26 October 2017 18:01

ну я не настаивал остальным вроде пофиг было

javascript_ru

26 October 2017 18:00

Это из-за $ в начале?

javascript_ru

26 October 2017 18:00

Значит, там все неправильно сделаны :)

javascript_ru

26 October 2017 17:57

Если мы нигде его не используем как fluent, то и не нужно его таким делать.

javascript_ru

26 October 2017 17:57

Ну чот херня была, кажись.

javascript_ru

26 October 2017 17:55

ibxth
А если все-равно компилировать, то чем flow лучше ts?

javascript_ru

26 October 2017 18:06

Да вы упоролись чтоли.

javascript_ru

26 October 2017 18:06

почему один докер на все а не 3 докера?

javascript_ru

26 October 2017 18:06

а че вы как-то не по пацански сделали

javascript_ru

26 October 2017 18:05

Я бы сделал RUN npm run build отдельной строчкой

javascript_ru

26 October 2017 18:03

Были ж какие-то проблемы — там заглавная буква, там нет.

javascript_ru

26 October 2017 18:02

Да, пусть будет. Заодно изучу Flow, дело полезное.

javascript_ru

26 October 2017 18:02

Надо было настаивать, пока была возможность 😆

javascript_ru

26 October 2017 18:01

Ты не подумай, к тебе никаких претензий. Ты сделал как было :)

javascript_ru

26 October 2017 18:01

Ок, измению на console.

javascript_ru

26 October 2017 18:00

Думаю, лучше убрать эти доллары.

javascript_ru

26 October 2017 18:00

Сделай свой правильно, остальные я отдельно подфикшу.

javascript_ru

26 October 2017 18:00

Насчет ридми, чем sh с npm run build отличается от других блоков?

javascript_ru

26 October 2017 17:57

Я считаю, что fluent API это вполне ок, но в меру.

javascript_ru

26 October 2017 17:56

Насколько я знаю, хорошая практика возвращать this вместо void, чтобы можно было делать цепочки.

В bots/jabber/bot.js было return client.sendMessage(...), но я не знаю точно что возвращает этот метод. Вроде, void, но его ответ не используется все равно.

javascript_ru

26 October 2017 17:54

Я там откомментил в PR, прошу всех заинтересованных лиц обратить внимание.