一、先看贡献指南
它会按自己的习惯改,但项目有自己的规范。先让它读 CONTRIBUTING.md。二、改动要小
一个大 PR 没人 review。拆成几个小 PR,每个解决一个问题。三、测试要过
项目 CI 有格式检查,它写的代码功能对但风格不符,CI 直接挂。三件事都是常识,但一开始完全没想到。
现在给开源项目提 PR 之前,固定让它读三个文件:贡献指南、最近的 commit 风格、一个已合并的 PR。
主楼- 暂无回复
继续逛逛
一、先看贡献指南
它会按自己的习惯改,但项目有自己的规范。先让它读 CONTRIBUTING.md。
二、改动要小
一个大 PR 没人 review。拆成几个小 PR,每个解决一个问题。
三、测试要过
项目 CI 有格式检查,它写的代码功能对但风格不符,CI 直接挂。
三件事都是常识,但一开始完全没想到。
现在给开源项目提 PR 之前,固定让它读三个文件:贡献指南、最近的 commit 风格、一个已合并的 PR。