• 请不要在回答技术问题时复制粘贴 AI 生成的内容
phx13ye
V2EX  ›  程序员

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

  •  
  •   phx13ye ·
    Alwayswithme · Apr 21, 2015 · 5890 views
    This topic created in 4215 days ago, the information mentioned may be changed or developed.
    一行代码少了个加号, 用pull request会不会太奇怪
    Supplement 1  ·  Apr 21, 2015
    Excited!
    已经提交并cherry pick了
    21 replies  •  2015-04-21 18:02:47 +08:00
    jadecoder
    jadecoder
    Apr 21, 2015❤️1
    1
    代码错了,那结果肯定也错了,你就提就好,混个contribution
    kasp
    kasp
    Apr 21, 2015❤️1
    2
    曾经给phpwind发邮件提示变量拼写错误。。
    phx13ye
    phx13ye
    OP
    Apr 21, 2015
    3
    我提了,还签了个Individual Contributor License Agreement v1.0
    感觉好高兴的样子
    rock_cloud
    rock_cloud
    Apr 21, 2015❤️1
    4
    曾给UFLDL的中文翻译版作者发邮件说公式推导错误。。
    还复习了一下微积分=。=
    zenliver
    zenliver
    Apr 21, 2015❤️1
    5
    发现了就提呗, 既能做贡献, 又能刷个contribution
    cbsw
    cbsw
    Apr 21, 2015❤️1
    6
    这种情况我一般提 issuse
    phx13ye
    phx13ye
    OP
    Apr 21, 2015
    7
    @rock_cloud 原版就是错的吗?
    bombless
    bombless
    Apr 21, 2015❤️1
    8
    有了第一次就容易来第二次,所以不要嫌贡献小,大胆提吧。
    bombless
    bombless
    Apr 21, 2015
    9
    @phx13ye 是签好之后拍照发图片吗?
    phx13ye
    phx13ye
    OP
    Apr 21, 2015
    10
    @bombless 是一个google form生成的条款
    bombless
    bombless
    Apr 21, 2015
    11
    @phx13ye 但是不是要签名么……难道我理解错了。
    是 Apache 项目么?
    bombless
    bombless
    Apr 21, 2015
    13
    @phx13ye 谢谢!
    原来是这样,这体验和信用卡付款差不多嘛。

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

    于是我决定今年办个护照(逃
    efi
    efi
    Apr 21, 2015❤️1
    14
    点中行数,加个code comment
    rock_cloud
    rock_cloud
    Apr 21, 2015
    15
    @phx13ye 英文原版里没有公式推导,中文版里的推导是翻译者加的,其实我纠正的那个错误应该是翻译者的笔误。
    xiang880401
    xiang880401
    Apr 21, 2015❤️1
    16
    @efi 别人的项目也可以加 comment 吗?
    learnshare
    learnshare
    Apr 21, 2015❤️1
    17
    也有朋友给我提过一个错别字的 PR,多好的贡献。
    crystaldust
    crystaldust
    Apr 21, 2015❤️1
    18
    果断提啊,之前用一个第三方的库一直导致内存泄露,最后发现是代码里少了一个下划线。错误小不代表不严重啊
    phx13ye
    phx13ye
    OP
    Apr 21, 2015
    19
    Excited!
    已经cherry pick了
    2hf
    2hf
    Apr 21, 2015
    20
    congratulations~
    sivacohan
    sivacohan
    Apr 21, 2015
    21
    我给saltstack提了个pull request,把false改成None
    About   ·   Help   ·   Advertise   ·   Blog   ·   API   ·   FAQ   ·   Privacy   ·   Solana   ·   4933 Online   Highest 6679   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 124ms · UTC 07:16 · PVG 15:16 · LAX 23:16 · JFK 02:16
    ♥ Do have faith in what you're doing.