These are chat archives for BicycleStudio/Ewe

1st
Nov 2015
Sergey Vaytsel
@Telhar
Nov 01 2015 04:41
По ревью с DSound посмотрел, написал пару комментов, надо поправить и, может, обсудить.
На счёт последнего сообщения - это всё потому, что в ревью по DSound попали изменения файлов IO, ThreadManager, OpenGLFacade и так далее. Это очень нехорошо, в дальнейшем, если возникает такая потребность, необходимо сделать следующее:
  • зафиксировать изменения в текущей ветке Х
  • перейти на мастера
  • сделать ветку Y и перейти на неё
  • сделать изменения в левых файлах
  • выложить на маленькое ревью, изменения в котором пройдут в мастер с полпинка
  • выкачать свежий мастер
  • перейти на ветку X
  • сделать rebase на мастера
Остальные ревью посмотрю по возможности, может вечером.

глянул мельком. надо закрыть их, т.к. туда попали изменения из ветки с Sound.

  • закрыть ревью по sound
  • сделать rebase каждой ветки на мастера
  • создать пул-реквесты

тогда там уже не будет лишнего.

Sergey Vaytsel
@Telhar
Nov 01 2015 08:38
сделал в вафле еще один столбик - Necessary задачи, как бы первостепенные, важные, но после Ready задач)
Sergey Vaytsel
@Telhar
Nov 01 2015 09:05
слушай, а мы с тобой не рассматривали вопрос использования баз данных. уместно ли?
Valentin Zaborcev
@Swipes13
Nov 01 2015 09:06
не думаю, что бд уместно. но обсудить можно :)
Valentin Zaborcev
@Swipes13
Nov 01 2015 09:42
ревью по Sound готово. там только вопрос с тем, что тебе не понравилось чтение из файла...
остальное смотри внимательно :)
Sergey Vaytsel
@Telhar
Nov 01 2015 10:08
да не то чтобы не понравилось)
просто везде используем STL, а тут C-шный способ, как то не канонично.
но, вообще говоря, не принципиально.
Valentin Zaborcev
@Swipes13
Nov 01 2015 12:53
#49 pr залит, следующий pr так же зависит от этого
Sergey Vaytsel
@Telhar
Nov 01 2015 13:01
а в чём зависимость?
Valentin Zaborcev
@Swipes13
Nov 01 2015 13:02
ну смотри, была ветка звук. я её доделал, от неё сделал ветку графикаЛог, сделал логирование и от логирования (а не от звука) сделал ещё одну ветку) потому последовательно
Sergey Vaytsel
@Telhar
Nov 01 2015 13:04
вот тебе challenge: пожонглируй ветками и коммитами так, чтобы эти ветки были отделены друг от друга)
заодно с git попрактикуешься)
Valentin Zaborcev
@Swipes13
Nov 01 2015 13:12
готово)
Sergey Vaytsel
@Telhar
Nov 01 2015 14:52
ночью посмотрю)
Sergey Vaytsel
@Telhar
Nov 01 2015 17:41
или завтра на работе время выкрою)