我的 PR 被要求改三遍
第一次给开源项目提 PR,维护者很耐心,但我的提交已经被打回两遍了:先是代码风格,再是测试覆盖。
有点受挫,但也想知道:这是正常的吗?第三遍该怎么改才能一次过?
第一次给开源项目提 PR,维护者很耐心,但我的提交已经被打回两遍了:先是代码风格,再是测试覆盖。
有点受挫,但也想知道:这是正常的吗?第三遍该怎么改才能一次过?
太正常了,改三遍说明维护者愿意带你。最怕的是那种看都不看直接关掉的。
应对就一条:把每一条意见当成一次学习机会,改完在评论区说清楚「我改了什么、为什么这么改」,维护者会觉得你靠谱,第三次基本就过了。
把维护者的意见拆成清单,逐条对照改,改一条勾一条。提交信息里写清楚改动点,能省掉很多来回。