| Files | Last commit | Last update |
|---|---|---|
refactor(pr): share NormalizeLabels, dedup pr label --add, drop dead SetPRLabels (#498) #498 三项收口(#475 评审 P2 跟踪): 1. pr label --add trim/dedup:把 edit 包的 normalizeLabels 提升为 cmdutil.NormalizeLabels (split+trim+dedup+empty,幂等兼容 cobra 预切),pr label --add 与 pr edit 共用; --add 无有效标签名时返回 usage error。消除 pr label 的 strings.Split 裸切。 2. 删除 SetPRLabels 死代码:全树无调用方(pr edit 走 UpdatePROptions.Labels, pr label 用 AddLabelsToPR/RemoveLabelFromPR)。避免未经验证的导出函数误导。 3. docs/COMMANDS.md 自动推断名单补 pr label/edit/close/ready(均调 ResolveRepo)。 - pkg/cmdutil/text_file.go: 新增 NormalizeLabels + 9 用例测试 - pkg/cmd/pr/edit/edit.go: normalizeLabels→cmdutil.NormalizeLabels,删本地实现,去 strings import - pkg/cmd/pr/label/label.go: --add 用 cmdutil.NormalizeLabels + 空值拒绝 - api/queries_pr.go: 删 SetPRLabels - docs/COMMANDS.md: 自动推断名单补 pr label/edit/close/ready | 2 months ago | |
test(pr): cover pr label --add empty-after-normalize usage error (review P2) 评审 P2:pr label --add 规范化后无有效标签名→usage error 的集成路径无测试。 补 TestLabelRunAddEmptyAfterNormalizeDoesNotMutate( / ',' / ' , , ' 不触发远端写、exit2)。 | 2 months ago |
| Files | Last commit | Last update |
|---|---|---|
| 2 months ago | ||
| 2 months ago |