掘金上居然有文章教人怎么给开源软件提 typo issue

[cwp374240920]

原文链接，评论区一片叫好声说学会了这就去提... 个人觉得文章可能只是破事水，没啥产出硬产出提高曝光度，但是这也在一定意义上教会了某些人去提这种垃圾 issue，给作者徒增了不少破事，如何评价这种行为呢？ PS：作者不会是那个公司的吧？

[jxlwqq]

新手从 fix typo“入门”开源项目，是一个比较容易上手的选择。提 pr 的时候可以顺便熟悉一下社区的文化和贡献流程。

[Rainshaw]

@cwp374240920 如果污了您眼了我很抱歉，建议 block 我，谢谢。改 typo 并不是无效 issue 。对于一个初学者，无论是字词 typo 还是其它 typo 都会很影响其对于项目的第一感觉，并且提个 typo 让文档更加规范在我看来也是很好的。项目核心开发者可能没有太多精力去进行校对工作，那即便他们是为了刷 pr，为了增加简历内容，提 PR 来完善文档我认为对于社区也是有帮助的。且，不会真有 HR 直接把提 PR 当闪光点吧，非核心开发者就算是提 PR 修了几个 bug 也只能显示他熟悉 git 流程，对于其”闪光面“我认为无任何帮助。

@wszgrcy 哎，代码格式有问题也可以提 pr 哇，比如之前没有用过 eslint 或者 black，提个 PR 整理一下格式也没啥，主要还是看 PR 的内容。如果 PR 里就是单纯的加个回车啥的那确实恶心人。

[KUROE]

@Rainshaw #13 不要混淆概念，这和国不国人有什么关系？小女孩捡到五块钱交给警察叔叔，发在网上肯定会被广泛称赞。你一个二三十的大老爷们还想靠这种事提升自己的形象，丢不丢人啊？流汗黄豆是吧😅😅😅

[jxlwqq]

任何一个对代码库有利的 PR 都是值得提交的，包括修复代码中的 typo 或者静态检查错误。

[hyy1995]

哈哈哈，点进来之前我就猜是不是这个帖了，我昨天看到的，一阵不适

[vain]

@Rainshaw 如果这小女孩的行为是为了混一个好看的简历，那这行为就该批评。否则，一个小女孩能看懂代码，还能参与其中，值得鼓励。

[whywaoxaks]

这事儿也不伤天害理，自己干了就干了，吹牛逼的时候，悄悄跟人说自己是某项目的贡献者，这就得了。
但让人恶心的是，他还把这当做一件莫大的光荣，到处跟人翩，恨不得写在他家家谱里。

[Rainshaw]

@vain 为了混好看的简历，也不应该批评。而是如果其 PR 内容是纯粹没事找事，该批评，如果确实修复了 typo 就毫无问题。

[leo108]

@acmore #10 你不捡我不捡，火箭首席设计师会去捡？最终还不得雇佣全职的清洁工？

开源项目会请人专门来改 typo ？

[learningman]

@Rainshaw 如果是国内的小女孩的话也会赞扬的。因为修改这个并完成整个流程，对于一个小女孩来说本身是一件困难的事情。
但是一个成年人整天关注这种东西，好像就已经不太对了。

[Pastsong]

改 typo 发 PR 比某校先提交漏洞再自己修掉刷 paper 要好的多吧，那一种才是真正浪费开发者时间

[ayase252]

一个开源项目如果有人会花费 ta 自己的时间来修 typo 。维护者应该感恩，更不会认为这是所谓的垃圾 /无效 PR 。

很多人的第一个 PR 就是修 typo，对于提交者而言，这是破除「心魔」的一次好机会。并不是你要会完美的添加一个模块或者接口才有资格向开源项目提交 PR，正如前面所说的，任何能够改善代码库的 PR 都可以。对于维护者而言，typo 通常是很难注意到的，修掉 typo 一般不会改变软件功能，review 起来也很容易，风险很低，为什么不接受呢？

[Rainshaw]

@learningman 我持不同意见，应该是对初学者来讲是困难的事而不是小女孩，因为在熟练的开发者看来，不过是点几下鼠标，输入几个字符而已。成年人也可以通过提 typo 来破除 @ayase252 所说的”心魔“。


对楼上持反对意见者：
”诸君的辫子长在心里“，认为提 typo 实在是太简单、太”蠢“了，不能彰显技术水平，却能和其他功能代码贡献者享有同等地位——“Contributor”标签——这是一种傲慢。

[xuboying]

这是教人类来做工具的事情么。开发都和工具卷起来了。。。

[xiadong1994]

这种 typo 的 PR 有人写 bot 全自动地提。fix typo 我觉得没什么，但是只 fix typo 并且动机根本就不是想对项目有贡献就有点……但是一般的项目作者都不会去这么 judge 别人的动机，至少表面上还是很友善的。

[xuboying]

正常思维不是有需求了才去看代码，顺手改掉 typo 么。改掉的应该也是自己 IDE 配了插件检查出来的。

[learningman]

@Rainshaw #33 https://elinux.org/images/0/06/Lca13-gregkh-I-dont-want-your-code.pdf
请先看一下这篇文章
就不讨论动机是不是不纯了，这种 pr 本身带来的收益，也不能说有多大。

[Rainshaw]

@learningman 太长。。。但这种 PR 在 Github 上只需要负责 merge 的人，点两下鼠标即可，所带来的开销也是很小的。

[learningman]

@Rainshaw #38 但是如果是 100 个这样的 PR 呢，你可以看看 Vue 的 PR 列表里面有多少个活动的 PR，这种无意义的 PR 还是好一点为佳。

[acmore]

@leo108 我想我已经说得很清楚了，爱捡就捡。
但以此把不属于自己的成就挂在自己头上就是无耻了。