Redundant commitConvention and repoType added to .all-contributorsrc file on add
维护者通常 1 天内回复
@lwasser 已经在做这个了。
开始于 2026年10月27日。
评估
- 难度
- 2/5
- 预计耗时
- 1-3 小时
- 新手友好度
- 45/100
- Issue 类型
- 缺陷
- 描述清晰度
- 描述清楚
- 活跃度
- 停滞
- 技术栈
- javascript
- 领域
- cli
调研方向
首先使用一个最小的 .all-contributorsrc 和 npx all-contributors-cli add JoshuaKGoldberg doc 重现该问题。跟踪 add 命令的配置更新路径，并检查文档中所述的默认值是如何写入的。完成的标准是：不包含这些属性的现有文件在执行命令后保持不变，并且为该重现用例提供覆盖率。
由索引模型根据 Issue 内容生成。
描述
Describe the bug
.all-contributorsrc files shouldn't need to include commitConvention or repoType properties. They both have default values as documented on https://allcontributors.org/docs/en/bot/configuration:
commitConvention:"angular"repoType:"github"
But, running npx all-contributors-cli add ... is adding commitConvention or repoType properties with their default values to .all-contributorsrc files that don't have them explicitly.
{
"contributors": [ ... ],
"projectName": "repros-example",
"projectOwner": "JoshuaKGoldberg",
+ "repoType": "github",
+ "commitConvention": "angular"
}
To Reproduce
See https://github.com/JoshuaKGoldberg/repros/tree/all-contributors-cli-add-field-defaults.
- Create a repo with a minimal
.all-contributorsrc - Run
npx all-contributors-cli add JoshuaKGoldberg doc
Expected behavior
The .all-contributorsrc file shouldn't have commitConvention or repoType added, since they have documented defaults.
Additional context
I filed a bunch of feature requests over on https://github.com/all-contributors/all-contributors for more default-friendly behavior. https://github.com/all-contributors/all-contributors/issues/808 is something of a hub for them.
- 主要语言
- JavaScript
- 星标
- 425
- 派生
- 152
- 平均合并
- 1 天 6 小时
- 30 天内合并 PR
- 17
环境准备
- 没有 Dockerfile 或 Docker Compose 文件
- 有 Pull Request 模板
- 阅读贡献指南
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
all-contributors/cli 的其他 Issue
-
Unquoted commit message is causing `npx all-contributors add` to fail on v7.0.0可能已有人在做 @stchigel 于 2 天前认领。 未关闭type: bug
难度 2/5 1-3 小时 新手友好度 78/100
all-contributors/cli#587 · 1 条评论 · 已指派 1 人 ·
维护者通常 1 天内回复
-
Invalid inquirer API access when running `npx all-contributors init` with v7.0.0可能已有人在做 @nhktmdzhg 于 1 天前认领。 未关闭regression type: bug
难度 2/5 1-3 小时 新手友好度 76/100
all-contributors/cli#586 · 1 条评论 · 1 个 reaction ·
维护者通常 1 天内回复
-
难度 4/5 3-5 天 新手友好度 35/100
all-contributors/cli#602 · 1 条评论 ·
维护者通常 1 天内回复
-
status: accepted type: infrastructure
难度 5/5 一周以上 新手友好度 30/100
all-contributors/cli#600 · 6 条评论 ·
维护者通常 1 天内回复
-
🔧 Tooling: Migrate to `pnpm`可能已有人在做 @michaelfaith 今天认领。 未关闭status: accepted
难度 3/5 1-2 天 新手友好度 52/100
all-contributors/cli#594 · 1 个 reaction ·
维护者通常 1 天内回复
查看 all-contributors/cli 的全部 Issue
相似的 Issue
-
难度 2/5 1-3 小时 新手友好度 76/100
obsidianmd/obsidian-developer-docs#281 ·
维护者通常 4 天内回复
-
bug
难度 2/5 1-3 小时 新手友好度 78/100
FriendsOfFlarum/synopsis#16 ·
-
难度 2/5 1-3 小时 新手友好度 78/100
Fei-Away/Codex-Dream-Skin#449 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 78/100
sysprog21/codetrial#604 · 1 条评论 ·
维护者通常 1 天内回复
-
bug
难度 2/5 1-3 小时 新手友好度 68/100
outline/outline#14489 · 1 条评论 ·
维护者通常 1 天内回复