AO
Кроме тех кто вообще не хочет фиксить свои патчи
Size: a a a
2020 June 25
AO
Знаю я несколько китайцев...
Dv
Кроме тех кто вообще не хочет фиксить свои патчи
За них приходится фиксить мне!
Dv
Из-за этого в WPF-Math некоторые PR'ы висят по несколько лет.
AO
Засылают крутой PR, там тысячи строк изменений. И он висит месяцами потому что говнокод.
c
Minoru
@a1batross: работать-то работает, только ревьюверам становится сложнее вспомнить, что они просили исправить. А так листаешь патч, смотришь на свои комменты, смотришь на исправленный код, и отмечаешь то, что починено
@a1batross: работать-то работает, только ревьюверам становится сложнее вспомнить, что они просили исправить. А так листаешь патч, смотришь на свои комменты, смотришь на исправленный код, и отмечаешь то, что починено
FO
Засылают крутой PR, там тысячи строк изменений. И он висит месяцами потому что говнокод.
Самое главное потом самому через пару месяцев не отревьюить свой собственный код, назвав аффтара неприличным словом :D
Dv
Самое главное потом самому через пару месяцев не отревьюить свой собственный код, назвав аффтара неприличным словом :D
Постоянно так делаю.
Dv
Часто ругаю какой-то код, чтобы обнаружить, что его изначальным автором являюсь я сам.
Dv
Это такая модернистская форма самобичевания, не иначе.
FO
Постоянно так делаю.
Проблема в том, что я ревью таки отправил.... Народ в репе был в шоке )
AO
За них приходится фиксить мне!
Ну когда засылают какую-нибудь фичу это ещё можно.
А какой-нибудь специфичный порт, вон висит порт движка на WinRT, это сложно. Мне негде проверить, а принимать нерабочий код даже собственноручно переделанный под гайды проекта не хочется. Ну, я так делал и оказалось это превращает кодовую базу в помойку.
А какой-нибудь специфичный порт, вон висит порт движка на WinRT, это сложно. Мне негде проверить, а принимать нерабочий код даже собственноручно переделанный под гайды проекта не хочется. Ну, я так делал и оказалось это превращает кодовую базу в помойку.
AO
Поэтому у меня сейчас требование -- пилите порт, делайте автосборку на CI.
Dv
Проблема в том, что я ревью таки отправил.... Народ в репе был в шоке )
Действительно не считаю это проблемой до тех пор, пока такой фидбэк конструктивный. Если ты видишь способ улучшить какой-то код — без разницы, старый или новый — хорошо бы эту информацию донести до того, кто может это сделать.
AO
Если собирается, значит работает.
%)
%)
Dv
Поэтому у меня сейчас требование -- пилите порт, делайте автосборку на CI.
Вполне хороший подход, я думаю.
AO
Minoru
@a1batross: работать-то работает, только ревьюверам становится сложнее вспомнить, что они просили исправить. А так листаешь патч, смотришь на свои комменты, смотришь на исправленный код, и отмечаешь то, что починено
@a1batross: работать-то работает, только ревьюверам становится сложнее вспомнить, что они просили исправить. А так листаешь патч, смотришь на свои комменты, смотришь на исправленный код, и отмечаешь то, что починено
Про память это точно. Я обычно предполагаю что память у меня (и у других кодеров) как у рыбки.
FO
Поэтому у меня сейчас требование -- пилите порт, делайте автосборку на CI.
а где мне брать Power9 CI с компиляторами и либами от IBM?(:
AO
а где мне брать Power9 CI с компиляторами и либами от IBM?(:
Кросскомпилировать?
FO
компилятор от IBM не умеет в кросскомпиляцию