这是一个创建于 42 天前的主题，其中的信息可能已经有所发展或是发生改变。
最近在研究 agentscope ，跑一些例子代码时发现点问题，尝试自己修复了代码直接提 PR 。我是开源贡献者小白，想问下图里大佬回复后，后续该做什么我没明白。是没通过审核还是另外有关联的地方一起修复？请教各位大佬们分享分享经验。相关 PR 的地址 ( https://github.com/agentscope-ai/agentscope/pull/665)
 |
1
avrillavigne 42 天前
 |
 |
2
sthwrong 42 天前
翻译下回复就知道了，已经有解决方案了，你看到的可能是旧教程或者示例。
|
 |
3
MIUIOS 42 天前
他的意思很明确了，你看的文档是老的导致报的错误，实际这个问题已经解决过了的
|
 |
4
UnluckyNinja 42 天前  3
一般是这个流程：
在尽量不与现有 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 被接收了，所以对正式点的流程不太清楚。感谢大佬详细的解答，学习！我再研究下审核员提到的代码。
|
Select Language