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

V2EX = way to explore

V2EX 是一个关于分享和探索的地方

现在注册

已注册用户请登录

这是一个创建于 3511 天前的主题，其中的信息可能已经有所发展或是发生改变。

一行代码少了个加号, 用pull request会不会太奇怪

第 1 条附言 · 2015-04-21 15:57:25 +08:00

Excited!
已经提交并cherry pick了

加号

开源

代码

21 条回复 • 2015-04-21 18:02:47 +08:00

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 就在上面写了点个人信息
https://docs.google.com/forms/d/1GKuaUw10DfPzPy_XBnp3Wk_OLYnA4-Kg16wSiUkO3-A/viewform?formkey=dHBjc1YzdWhsZERUQnhlSklsbG1KT1E6MQ

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~

sivacohan

2015-04-21 18:02:47 +08:00

我给saltstack提了个pull request，把false改成None