AndrewHenry
V2EX  ›  GitHub

github 提 Pull requests 一次性写少一点的代码更容易被管理者接受,还是多一点更容易接受呢?

  •  1
     
  •   AndrewHenry · May 18, 2025 · 2901 views
    This topic created in 359 days ago, the information mentioned may be changed or developed.
    提 Pull requests 有啥提升通过率技巧吗?
    7 replies    2025-05-19 00:49:26 +08:00
    KagurazakaNyaa
        1
    KagurazakaNyaa  
       May 18, 2025
    质量高,结构清晰,文档齐全,侵入性低的代码容易被合并,最好风格和其它维护者保持一致
    renmu
        2
    renmu  
       May 18, 2025 via Android   ❤️ 3
    先提 issue
    whileFalse
        3
    whileFalse  
       May 18, 2025
    一次解决一个问题更容易接受
    craftsmanship
        4
    craftsmanship  
       May 18, 2025 via Android
    楼上老哥们说得对 从个人经历来看 遇到好的维护者 他可能会很耐心地告诉你怎么改 所以哪怕水平不是很够 在多次沟通修改之后 PR 也是有机会被接受的

    但提 PR 之前一定要先提 issue 跟维护者们沟通好你想改进的点 是不是人家需要的 得到对方的同意之后再动手提 PR 不然很可能你耗费了很多精力和热情 却吃个闭门羹 反过头来还想骂他们不识好歹😂
    MajestySolor
        5
    MajestySolor  
       May 18, 2025   ❤️ 1
    先提 issue ,充分交流
    不提 issue 直接 pr 的懒得看,至少我就是这样🤣
    yylxbiubiu
        6
    yylxbiubiu  
       May 18, 2025
    一次一个特性呀 清楚的描述这次修改做了什么修改、进行了哪些测试 跟代码量多少没关系吧
    foolishcrab
        7
    foolishcrab  
       May 19, 2025 via iPhone
    现在基本只有起步阶段的项目的 good first issue 比较容易合了,其他稳定下来的基本都要花非常长的时间融入社区。基本上是一个逐步建立 reputation 的过程,跟上班的代码一样的。
    而且这两年开源社区甚至越来越封闭了,挺没意思的
    About   ·   Help   ·   Advertise   ·   Blog   ·   API   ·   FAQ   ·   Solana   ·   4485 Online   Highest 6679   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 53ms · UTC 10:11 · PVG 18:11 · LAX 03:11 · JFK 06:11
    ♥ Do have faith in what you're doing.