看开源项目源码发现一行代码错了, 好提 PR 吗?

2015-04-21 14:18:24 +08:00
 phx13ye
一行代码少了个加号, 用pull request会不会太奇怪
4698 次点击
所在节点    程序员
21 条回复
jadecoder
2015-04-21 14:22:10 +08:00
代码错了,那结果肯定也错了,你就提就好,混个contribution
kasp
2015-04-21 14:25:30 +08:00
曾经给phpwind发邮件提示变量拼写错误。。
phx13ye
2015-04-21 14:31:43 +08:00
我提了,还签了个Individual Contributor License Agreement v1.0
感觉好高兴的样子
rock_cloud
2015-04-21 14:35:01 +08:00
曾给UFLDL的中文翻译版作者发邮件说公式推导错误。。
还复习了一下微积分=。=
zenliver
2015-04-21 14:38:50 +08:00
发现了就提呗, 既能做贡献, 又能刷个contribution
cbsw
2015-04-21 14:39:10 +08:00
这种情况我一般提 issuse
phx13ye
2015-04-21 14:39:20 +08:00
@rock_cloud 原版就是错的吗?
bombless
2015-04-21 14:40:24 +08:00
有了第一次就容易来第二次,所以不要嫌贡献小,大胆提吧。
bombless
2015-04-21 14:41:42 +08:00
@phx13ye 是签好之后拍照发图片吗?
phx13ye
2015-04-21 14:47:48 +08:00
@bombless 是一个google form生成的条款
bombless
2015-04-21 14:49:22 +08:00
@phx13ye 但是不是要签名么……难道我理解错了。
是 Apache 项目么?
phx13ye
2015-04-21 14:52:23 +08:00
bombless
2015-04-21 14:54:34 +08:00
@phx13ye 谢谢!
原来是这样,这体验和信用卡付款差不多嘛。

不过我没办过护照,要我填的话可能还很犹豫怎么填好。

于是我决定今年办个护照(逃
efi
2015-04-21 14:57:44 +08:00
点中行数,加个code comment
rock_cloud
2015-04-21 15:14:15 +08:00
@phx13ye 英文原版里没有公式推导,中文版里的推导是翻译者加的,其实我纠正的那个错误应该是翻译者的笔误。
xiang880401
2015-04-21 15:18:45 +08:00
@efi 别人的项目也可以加 comment 吗?
learnshare
2015-04-21 15:22:23 +08:00
也有朋友给我提过一个错别字的 PR,多好的贡献。
crystaldust
2015-04-21 15:49:28 +08:00
果断提啊,之前用一个第三方的库一直导致内存泄露,最后发现是代码里少了一个下划线。错误小不代表不严重啊
phx13ye
2015-04-21 15:56:43 +08:00
Excited!
已经cherry pick了
2hf
2015-04-21 17:56:45 +08:00
congratulations~

这是一个专为移动设备优化的页面(即为了让你能够在 Google 搜索结果里秒开这个页面),如果你希望参与 V2EX 社区的讨论,你可以继续到 V2EX 上打开本讨论主题的完整版本。

https://www.v2ex.com/t/185222

V2EX 是创意工作者们的社区,是一个分享自己正在做的有趣事物、交流想法,可以遇见新朋友甚至新机会的地方。

V2EX is a community of developers, designers and creative people.

© 2021 V2EX