#42772 Add dllehr-amd to CODEOWNERS and committers list

原始 PR 作者 dllehr-amd 合并时间 2026-05-21 05:10 文件变更 2 提交数 1 评论 3 代码增减 +8 / -8

执行摘要

为 AMD 集成添加新的代码审查者，并移除不再参与的人员。

可快速合并，注意 review 评论中的小建议（如迁移 gshtras 至 Emeritus Committers 可后续跟进）。

讨论亮点

gemini-code-assist[bot] 指出 CODEOWNERS 中 /vllm/v1/attention/ops/rocm*.py 路径遗漏了 @dllehr-amd，但实际 patch 显示该条目已包含 @dllehr-amd，评论可能基于中间版本。
mgoin 询问是否应将 gshtras 移至 Emeritus Committers 列表，但 PR 未处理。

在 .github/CODEOWNERS 中，将 ROCm 相关路径（如 /vllm/**/*rocm*、docker 等）的 owner 从 @gshtras @tjtanaa 替换为 @tjtanaa @dllehr-amd。
在 docs/governance/committers.md 中，将 AMD integration 的 committer 从 @gshtras 替换为 @dllehr-amd。

文件	模块	状态	重要度
`.github/CODEOWNERS`	CI/ 构建	modified	3.34
`docs/governance/committers.md`	文档	modified	1.32

分析完成后，这里会展示 LLM 生成的相对完整源码片段和详细注释。

CODEOWNERS 遗漏 dllehr-amd 在某路径上 other

gemini-code-assist[bot] 认为 /vllm/v1/attention/ops/rocm*.py 路径漏加了 dllehr-amd，但实际 patch 已包含。

结论：误报或基于旧版本，无需处理。 · 已解决

gshtras 是否应移至 Emeritus Committers other

mgoin 建议将 gshtras 迁移至 Emeritus Committers 列表以记录贡献。

结论：未采纳，仅简单替换。 · unresolved

低风险：仅修改元数据文件，不影响代码逻辑。需确保 CODEOWNERS 条目与实际 review 期望一致。

影响范围小：仅改变未来对 ROCm 相关 PR 的自动审查通知。团队工作流无实际变更。

当前没有检测到明确关联的 Issue 链接，后续同步到相关引用后会出现在这里。