Re: [讨论] 就算提早做完是不是不要回报比较好

楼主: leviliang (levi)   2023-04-20 05:39:22
看到这篇不禁让人想起从前的菜鸟时光
以前的确会聪明地压着一些东西
作为紧急时就可以马上拿出手的 buffer
不过 现在除非是处理紧急的 defects
否则我会尽可能地分割工作
一个 5-8 credits 的 user story
会切成大约 10 个 PR 来做
一天完成 1-2 个 PR
每个 PR 改动少、review 轻松、修正容易
剩下的时间就可以排 training、读 technical blog 或制作团队内小型技术分享的 work
shop
看到这里有些大大可能会开始觉得
这一定是在外商过太爽
你根本工作太闲
但有兴趣的朋友们可以试试看
刚开始不仅节奏会比你想像的要赶
切割 user story 更是没有想像中简单
(刚开始一天一个 PR 真的是要我命 还会不知不觉加班)
原本很直观一个 PR 解决的任务
要拆成两三个合理易懂的小 PR
这相当考验功力
如果对版控不熟悉
更容易弄巧成拙
花更多时间处理这些小分支
但好处也是直观的
以前习惯一个 PR 解决的东西
可能都会有十几二十几个档案的增改
现在降低到五六个
修改的程式行数大幅下降
琢磨细节更容易
程式可以写得更干净更有成就感
无论是可读性还是可维护性都是大大增加
除了技术上的好处外
软实力上我认为这帮助更大
第一就是切割工作的艺术
第二则是确实地量化并实践 Agile 中的 credits
不然每次在 backlog refinement 中
评估 US 的 credits 都凭直觉乱猜一通
经过了切割工作的“痛苦”磨练
现在我才真正有了透视 US 的感觉
作者: loadingN (sarsaparilla)   2023-04-20 07:35:00
上版的功能切干净,是本来就该做的,果然过太爽
作者: CoNsTaR ((const *))   2023-04-20 07:40:00
真的,每次看到那种一个 PR 上千行的看都不用看就知道一定菜鸟发的
作者: vi000246 (Vi)   2023-04-20 08:43:00
一个pizza切两块跟切十块的概念 不过做的事还是一样多
作者: yamagishi (山岸刑务官)   2023-04-20 08:52:00
code review 可以比较简单是真的,一次丢太多很难面面俱到
作者: naestnecniv (内湖金城武)   2023-04-20 09:38:00
但切完PR,review的速度比我PR发起的速度还慢就很麻烦了。
作者: Galbygene (sasori)   2023-04-20 13:14:00
请问PR是什么的缩写
作者: rabbitu04 (心日兔)   2023-04-20 13:18:00
pull request
作者: Galbygene (sasori)   2023-04-20 14:59:00
谢谢
作者: s06yji3 (阿南)   2023-04-20 15:05:00
Review通常都超拖的啊
作者: orangelite (清新柳橙)   2023-04-20 18:52:00
有点好奇原po是写前端还是后端?我自己前端的经验一个页面就是一个 pr分很多 pr 画面不完全感觉很怪…
作者: foreverk (文艺青年)   2023-04-20 19:15:00
你的画面如果一页有10个新的元件,你发在一个pr内的话,review的人要嘛花很常时间看,或是分好几天看卡住你的pr,你切分好给人review同时你还能继续开发或是改其他review的pr
作者: s06yji3 (阿南)   2023-04-20 19:18:00
这样怎知道10个PR merge起来没有问题...不过通常一个元件就可以是一个PR了
作者: foreverk (文艺青年)   2023-04-20 19:25:00
也不用10个pr,关联性较高的合起来发三四个,都比一整包出去好吧
作者: s06yji3 (阿南)   2023-04-20 19:27:00
啊,应该说该break down的应该是task而不是PR
作者: safe (safe)   2023-04-20 19:44:00
适用场景:案子时程不赶、团队 junior 偏多
作者: Ekmund (是一只小叔)   2023-04-20 23:49:00
理想状态是这样 如果不是临时改意大利面这周五上的话
作者: holmes2136 (holmes)   2023-04-21 14:36:00
切得好还可以避免conflict 的机会Review的人也轻松多
作者: d0068267 (WizXu)   2023-04-21 16:06:00
好闻
作者: bobokeke (韭菜们安安)   2023-04-21 23:33:00
我自己习惯一个PR不多过4个档案或250-300行程式,除非是asset/config/generated files不过我常常一天连发10张PR,哈哈哈切PR是一门艺术
作者: assembler80   2023-04-25 00:13:00
PR修改的程式码太多,review的人会超痛苦,而且对方可能在设计上有一些问题,但都写那么多程式了,也不太好退掉他的PR,要他重写

Links booklink

Contact Us: admin [ a t ] ucptt.com