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