看开源项目源码发现一行代码错了, 好提 PR 吗? - V2EX
V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
请不要在回答技术问题时复制粘贴 AI 生成的内容
phx13ye
V2EX    程序员

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

  •  
  •   phx13ye
    Alwayswithme 2015-04-21 14:18:24 +08:00 5121 次点击
    这是一个创建于 3850 天前的主题,其中的信息可能已经有所发展或是发生改变。
    一行代码少了个加号, 用pull request会不会太奇怪
    第 1 条附言    2015-04-21 15:57:25 +08:00
    Excited!
    已经提交并cherry pick了
    21 条回复    2015-04-21 18:02:47 +08:00
    jadecoder
        1
    jadecoder  
       2015-04-21 14:22:10 +08:00   1
    代码错了,那结果肯定也错了,你就提就好,混个contribution
    kasp
        2
    kasp  
       2015-04-21 14:25:30 +08:00   1
    曾经给phpwind发邮件提示变量拼写错误。。
    phx13ye
        3
    phx13ye  
    OP
       2015-04-21 14:31:43 +08:00
    我提了,还签了个Individual Contributor License Agreement v1.0
    感觉好高兴的样子
    rock_cloud
        4
    rock_cloud  
       2015-04-21 14:35:01 +08:00   1
    曾给UFLDL的中文翻译版作者发邮件说公式推导错误。。
    还复习了一下微积分=。=
    zenliver
        5
    zenliver  
       2015-04-21 14:38:50 +08:00   1
    发现了就提呗, 既能做贡献, 又能刷个contribution
    cbsw
        6
    cbsw  
       2015-04-21 14:39:10 +08:00   1
    这种情况我一般提 issuse
    phx13ye
        7
    phx13ye  
    OP
       2015-04-21 14:39:20 +08:00
    @rock_cloud 原版就是错的吗?
    bombless
        8
    bombless  
       2015-04-21 14:40:24 +08:00   1
    有了第一次就容易来第二次,所以不要嫌贡献小,大胆提吧。
    bombless
        9
    bombless  
       2015-04-21 14:41:42 +08:00
    @phx13ye 是签好之后拍照发图片吗?
    phx13ye
        10
    phx13ye  
    OP
       2015-04-21 14:47:48 +08:00
    @bombless 是一个google form生成的条款
    bombless
        11
    bombless  
       2015-04-21 14:49:22 +08:00
    @phx13ye 但是不是要签名么……难道我理解错了。
    是 Apache 项目么?
    bombless
        13
    bombless  
       2015-04-21 14:54:34 +08:00
    @phx3ye 谢谢!
    原来是这样,这体验和信用卡付款差不多嘛。

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

    于是我决定今年办个护照(逃
    efi
        14
    efi  
       2015-04-21 14:57:44 +08:00   1
    点中行数,加个code comment
    rock_cloud
        15
    rock_cloud  
       2015-04-21 15:14:15 +08:00
    @phx13ye 英文原版里没有公式推导,中文版里的推导是翻译者加的,其实我纠正的那个错误应该是翻译者的笔误。
    xiang880401
        16
    xiang880401  
       2015-04-21 15:18:45 +08:00   1
    @efi 别人的项目也可以加 comment 吗?
    learnshare
        17
    learnshare  
       2015-04-21 15:22:23 +08:00   1
    也有朋友给我提过一个错别字的 PR,多好的贡献。
    crystaldust
        18
    crystaldust  
       2015-04-21 15:49:28 +08:00   1
    果断提啊,之前用一个第三方的库一直导致内存泄露,最后发现是代码里少了一个下划线。错误小不代表不严重啊
    phx13ye
        19
    phx13ye  
    OP
       2015-04-21 15:56:43 +08:00
    Excited!
    已经cherry pick了
    2hf
        20
    2hf  
       2015-04-21 17:56:45 +08:00
    congratulations~
    sivacohan
        21
    sivacohan  
    PRO
       2015-04-21 18:02:47 +08:00
    我给saltstack提了个pull request,把false改成None
    关于     帮助文档     自助推广系统     博客     API     FAQ     Solana     974 人在线   最高记录 6679       Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 26ms UTC 22:32 PVG 06:32 LAX 14:32 JFK 17:32
    Do have faith in what you're doing.
    ubao msn snddm index pchome yahoo rakuten mypaper meadowduck bidyahoo youbao zxmzxm asda bnvcg cvbfg dfscv mmhjk xxddc yybgb zznbn ccubao uaitu acv GXCV ET GDG YH FG BCVB FJFH CBRE CBC GDG ET54 WRWR RWER WREW WRWER RWER SDG EW SF DSFSF fbbs ubao fhd dfg ewr dg df ewwr ewwr et ruyut utut dfg fgd gdfgt etg dfgt dfgd ert4 gd fgg wr 235 wer3 we vsdf sdf gdf ert xcv sdf rwer hfd dfg cvb rwf afb dfh jgh bmn lgh rty gfds cxv xcv xcs vdas fdf fgd cv sdf tert sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf shasha9178 shasha9178 shasha9178 shasha9178 shasha9178 liflif2 liflif2 liflif2 liflif2 liflif2 liblib3 liblib3 liblib3 liblib3 liblib3 zhazha444 zhazha444 zhazha444 zhazha444 zhazha444 dende5 dende denden denden2 denden21 fenfen9 fenf619 fen619 fenfe9 fe619 sdf sdf sdf sdf sdf zhazh90 zhazh0 zhaa50 zha90 zh590 zho zhoz zhozh zhozho zhozho2 lislis lls95 lili95 lils5 liss9 sdf0ty987 sdft876 sdft9876 sdf09876 sd0t9876 sdf0ty98 sdf0976 sdf0ty986 sdf0ty96 sdf0t76 sdf0876 df0ty98 sf0t876 sd0ty76 sdy76 sdf76 sdf0t76 sdf0ty9 sdf0ty98 sdf0ty987 sdf0ty98 sdf6676 sdf876 sd876 sd876 sdf6 sdf6 sdf9876 sdf0t sdf06 sdf0ty9776 sdf0ty9776 sdf0ty76 sdf8876 sdf0t sd6 sdf06 s688876 sd688 sdf86