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

42 天前
 Syriana

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

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

1811 次点击
所在节点    程序员
7 条回复
avrillavigne
42 天前
sthwrong
42 天前
翻译下回复就知道了,已经有解决方案了,你看到的可能是旧教程或者示例。
MIUIOS
42 天前
他的意思很明确了,你看的文档是老的导致报的错误,实际这个问题已经解决过了的
UnluckyNinja
42 天前
一般是这个流程:
在尽量不与现有 issue 重复的情况下,先针对问题提个 issue ,跟维护团队/合作者探讨下是什么类型的 issue ,原因是什么,是否有必要修复(确定完维护团队会给 issue 打一系列 tag ),这个阶段叫 triage ,一般需要最新版本的最小化复现,以便确定问题。
你可以同时在正文中表示愿意针对这个 issue 提交代码贡献,如果确定需要修复并且维护者愿意让你提交代码(也可能由于已有人在处理这个问题,就不需要你出力了),就进入敲代码提 PR 阶段。
PR 阶段会有人来 review ,双方确保最终代码正确、可维护、符合项目风格,如果没问题了,等人合并 PR 就结束了。
如果 triage 阶段确定问题已修复/是个误会/非计划,那就没必要提 PR 了,避免敲完代码后发现实际没有问题/工作重叠等问题。你现在就是遇到这种情况了😂,合作者说更可能是因为路径指向了过时的教程和文档,于是你可以:
1. 基于最新代码尝试复现,如果确定修复了,可以表示确实是个误会,新版本没有这个问题,然后关闭 pr 和 issue 就行了。
2. 如果你愿意更进一步,可以尝试解决路径过时的问题(在另外的 issue 和新的 PR 里完成)。
Syriana
41 天前
@sthwrong 嗯嗯,搜索代码发现有同样命令的另外个示例,但还在研究,没有跑通
Syriana
41 天前
@totoro52 好的,感谢大佬指导
Syriana
41 天前
@UnluckyNinja 好的,因为我之前在另外个项目里发现个 BUG ,直接提了 PR 被接收了,所以对正式点的流程不太清楚。感谢大佬详细的解答,学习!我再研究下审核员提到的代码。

这是一个专为移动设备优化的页面(即为了让你能够在 Google 搜索结果里秒开这个页面),如果你希望参与 V2EX 社区的讨论,你可以继续到 V2EX 上打开本讨论主题的完整版本。

https://ex.noerr.eu.org/t/1153217

V2EX 是创意工作者们的社区,是一个分享自己正在做的有趣事物、交流想法,可以遇见新朋友甚至新机会的地方。

V2EX is a community of developers, designers and creative people.

© 2021 V2EX