V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
phx13ye
V2EX  ›  程序员

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

  •  
  •   phx13ye ·
    Alwayswithme · 2015-04-21 14:18:24 +08:00 · 4638 次点击
    这是一个创建于 3489 天前的主题,其中的信息可能已经有所发展或是发生改变。
    一行代码少了个加号, 用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
    @phx13ye 谢谢!
    原来是这样,这体验和信用卡付款差不多嘛。

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

    于是我决定今年办个护照(逃
    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  
       2015-04-21 18:02:47 +08:00
    我给saltstack提了个pull request,把false改成None
    关于   ·   帮助文档   ·   博客   ·   API   ·   FAQ   ·   实用小工具   ·   2692 人在线   最高记录 6679   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 23ms · UTC 01:47 · PVG 09:47 · LAX 17:47 · JFK 20:47
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.