请教各位大佬,开源贡献小白尝试献提了个 PR 后,后续不知道该如何进行 - V2EX
V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
请不要在回答技术问题时复制粘贴 AI 生成的内容
Syriana
V2EX    程序员

请教各位大佬,开源贡献小白尝试献提了个 PR 后,后续不知道该如何进行

  •  
  •   Syriana 129 天前 2002 次点击
    这是一个创建于 129 天前的主题,其中的信息可能已经有所发展或是发生改变。

    最近在研究 agentscope ,跑一些例子代码时发现点问题,尝试自己修复了代码直接提 PR 。我是开源贡献者小白,想问下图里大佬回复后,后续该做什么我没明白。是没通过审核还是另外有关联的地方一起修复?请教各位大佬们分享分享经验。相关 PR 的地址 ( https://github.com/agentscope-ai/agentscope/pull/665)

    ( https://imgur.com/a/HfR3HhN)

    7 条回复    2025-08-19 08:38:39 +08:00
    avrillavigne
        1
    avrillavigne  
       129 天前
    sthwrong
        2
    sthwrong  
       129 天前
    翻译下回复就知道了,已经有解决方案了,你看到的可能是旧教程或者示例。
    MIUIOS
        3
    MIUIOS  
       129 天前
    他的意思很明确了,你看的文档是老的导致报的错误,实际这个问题已经解决过了的
    UnluckyNinja
        4
    UnluckyNinja  
       129 天前   3
    一般是这个流程:
    在尽量不与现有 issue 重复的情况下,先针对问题提个 issue ,跟维护团队/合作者探讨下是什么类型的 issue ,原因是什么,是否有必要修复(确定完维护团队会给 issue 打一系列 tag ),这个阶段叫 triage ,一般需要最新版本的最小化复现,以便确定问题。
    你可以同时在正文中表示愿意针对这个 issue 提交代码贡献,如果确定需要修复并且维护者愿意让你提交代码(也可能由于已有人在处理这个问题,就不需要你出力了),就进入敲代码提 PR 阶段。
    PR 阶段会有人来 review ,双方确保最终代码正确、可维护、符合项目风格,如果没问题了,等人合并 PR 就结束了。
    如果 triage 阶段确定问题已修复/是个误会/非计划,那就没必要提 PR 了,避免敲完代码后发现实际没有问题/工作重叠等问题。你现在就是遇到这种情况了,合作者说更可能是因为路径指向了过时的教程和文档,于是你可以:
    1. 基于最新代码尝试复现,如果确定修复了,可以表示确实是个误会,新版本没有这个问题,然后关闭 pr 和 issue 就行了。
    2. 如果你愿意更进一步,可以尝试解决路径过时的问题(在另外的 issue 和新的 PR 里完成)。
    Syriana
        5
    Syriana  
    OP
       128 天前
    @sthwrong 嗯嗯,搜索代码发现有同样命令的另外个示例,但还在研究,没有跑通
    Syriana
        6
    Syriana  
    OP
      nbsp;128 天前
    @totoro52 好的,感谢大佬指导
    Syriana
        7
    Syriana  
    OP
       128 天前
    @UnluckyNinja 好的,因为我之前在另外个项目里发现个 BUG ,直接提了 PR 被接收了,所以对正式点的流程不太清楚。感谢大佬详细的解答,学习!我再研究下审核员提到的代码。
    关于     帮助文档     自助推广系统     博客     API     FAQ     Solana     913 人在线   最高记录 6679       Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 33ms UTC 22:29 PVG 06:29 LAX 14:29 JFK 17:29
    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