- *programming *opensourceЕсть некий проект, лиддевом которого я являюсь. В него также коммитят код еще несколько людей. Пока люди коммитили добавления и исправления к тому, что в оригинале я писал, было легко ревьювить их коммиты и смотреть, а не наделают ли они фигни. Сейчас, когда один из них пилит IRC-клиент, полностью свой модуль и все такое, мне уже гораздо труднее вникать в семантику его кода, строить все взаимосвязи и все такое, тем более, что оно пока еще только разрабатывается. Что делать, жуйк? Я вижу два варианта:
1. Сказать «ты автор плагина, ты и ответственен за задницы в нем», а самому следить лишь за отсутствием ну совсем очевидной хуйни.
2. Сказать «пока пили, перед релизом заревьювим и оценим».
Первый вариант мне импонирует таки больше :3from Azoth_primary, 6 months ago
Replies (35)
- @magog:не поверишь, но мне тоже. Все равно самые места, в которых я менее всего уверен я все равно спрашиваю у тебяfrom Kopete, 6 months ago
- @k0st1x:как лид девелоперу, думаю, будет полезно проводить code review / test review своего продукта :) пусть даже плагинаfrom Work, 6 months ago
- @0xd34df00d:@k0st1x Второй вариант, типа?
Я и так смотрю, чо мержу к себе, но не особо слежу за смыслом — трудно это, когда изменения смысла разбросаны по куче коммитов и так неродного кода :)from web, 6 months ago, in reply to /5 - @0xd34df00d:@netneladno Даже в ПРЫЩАХ за каждую подсистему свой чувак ответственен.from web, 6 months ago, in reply to /7
- @netneladno:вообще в постгресе за год проводится 4 коммитфеста например
и каждый присланный патч ктото ревьюит и ставит резолюциюfrom E76CBB0D, 6 months ago - @k0st1x:@0xd34df00d значит проводи не ревью всего кода, а ревью архитектуры и тестов (как раз, чтобы понять архитектуру), причем не в конце а периодически в процессе; все таки ответственным для людей будешь именно тыfrom Work, 6 months ago, in reply to /8
- @0xd34df00d:@k0st1x Почти второе, ок. Просто каждый коммит ревьювить как-то ну очень неудобно.
И это, тесты гуи-приложений я таки ниасилил :3from web, 6 months ago, in reply to /15 - @mrmcc:надо делегировать. даже Сам не читает все коммиты в ядроfrom чорный проприетарный арм, 6 months ago
- @0xd34df00d:@magog Учитывая, что 90% кода ядра — драйвера и прочая хрень, которая не сильно влияет на ядро...from web, 6 months ago, in reply to /24
- @magog:@0xd34df00d да. Так и тут должно быть. Вообще было бы пиздато написать сантаксический анализатор стиля кода.from Kopete, 6 months ago, in reply to /26
- @magog:@0xd34df00d можешь и на нем. Тогда пропадет заеб с кодом.from Kopete, 6 months ago, in reply to /28
- @mrmcc:@0xd34df00d я слабо представляю, как линус что-то ревьювит в arch/arm/from чорный проприетарный арм, 6 months ago, in reply to /26
- @0xd34df00d:@mrmcc Вот и я про то же. Линусу остается ревьювить весьма малую часть кода. Вероятно, весьма тесно перекрывающуюся с тем, что он всегда сам писал тоже.from web, 6 months ago, in reply to /30
- @0xd34df00d:@Elemir Code & programming style не спасет от семантической хуиты более высокого уровня, очевидно же.from web, 6 months ago, in reply to /33
- @Minoru:
Первый вариант мне импонирует таки больше :3
И это правильно, я считаю.from netbook, 6 months ago