search

emotz / kladovka

Optimizer of inventory for games
http://kladovka.herokuapp.com/
MIT License
0 stars 0 forks source link

Dev #14

Closed ITarako closed 7 years ago

Eoksni commented 7 years ago

ммм то есть не ревьювить ?

Eoksni commented 7 years ago

и ты зря если честно пушишь новые изменения в тот же пуллреквест, надо было бы лучше старый закрыть потом уже новый начинать.

Eoksni commented 7 years ago

Все готово ? Ревьювить ? Или еще рано ?

ITarako commented 7 years ago

то что касается async db, там освоение промисов и минимум алгоритмики, и по сути нечего смотреть в mongo!! написана либа, но она еще не подключена к кладовке

Eoksni commented 7 years ago

Окей то есть не смотреть ?

Eoksni commented 7 years ago

И хватит в конце концов смешивать issues. Закончи правки по текущему пулл реквесту, потом принимайся за новые исусы. Если так не терпится их сделать - делай в отдельной ветке. Мы так никогда этот несчастный пулл реквест не закроем такими темпами.

ITarako commented 7 years ago

ой, все!!1

ITarako commented 7 years ago

C синхронной бд ничего не делал, ибо кладовка уже асинхронная и тесты для этой бд отдельные, синхронные. Может быть вообще уже выпилить её файлы