已关闭
refactor(discussions): DRY print helpers (org/project list+view 4 处复制) #500
aflyingto创建于  8月8日关闭于  8月8日
aflyingto成员
8月8日 创建

来源

PR #459(项目讨论 list+view)独立评审 P2:project list/view 的 printDiscussions/printDiscussion 是 org list/view 的逐字节复制(~33+24 行),未来格式变更需改 4 处。

现状

  • pkg/cmd/discussions/list/list.go printDiscussions
  • pkg/cmd/discussions/view/view.go printDiscussion
  • pkg/cmd/discussions/project/list/list.go printDiscussions(复制)
  • pkg/cmd/discussions/project/view/view.go printDiscussion(复制)

org 版本(PR #458)已合入未共享;project(PR #459)为一致性亦复制。Go 包边界使就地共享略 awkward。

建议

抽取到共享包(如 pkg/cmd/discussions/render 或 shared),4 个调用点统一引用。低优先级,非阻塞。

范围

scope/discussions, type/refactor, risk/low

likedislike
Aaflyingto成员
8月8日 添加了label:risk/lowscope/discussions, status/triage,type/refactor
Aaflyingto成员
8月8日 关联了pull request:refactor(discussions): DRY print helpers into render package
Aaflyingto成员
8月8日 关闭了 issue