V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
• 请不要在回答技术问题时复制粘贴 AI 生成的内容
Syriana
V2EX  ›  程序员

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

  •  
  •   Syriana · 20 天前 · 1646 次点击

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