app_installations reports an error on every repo or suborg change when the plugin is not configured
维护者通常 1 天内回复
还没有人认领这个 Issue。
评估
- 难度
- 2/5
- 预计耗时
- 1-3 小时
- 新手友好度
- 64/100
- Issue 类型
- 缺陷
- 描述清晰度
- 描述清楚
- 活跃度
- 活跃
- 技术栈
- javascript
- 领域
- backend
调研方向
从 lib/settings.js 开始，找到 syncSelectedRepos 在 hasChangedConfigs 为真时调用 syncAppInstallations 的位置，并定位需要使用企业认证客户端的地方。添加一个守卫，使插件在组织、子组织或仓库级别都不存在 app_installations 配置时提前返回，即使仓库发生了变化。完成标准是：修改了仓库文件、但既没有企业安装也没有 app_installations 配置的 PR 或 push 能够成功结束，并且配置存在时的原有路径仍然正常工作。
由索引模型根据 Issue 内容生成。
描述
Version: 3.0.0
syncSelectedRepos always calls syncAppInstallations with the changed repos (lib/settings.js). Because hasChangedConfigs is true, the "No app_installations config found, skipping" early return never applies. Without an enterprise installation, every PR or push that touches a repo file fails with:
Cannot sync app installations: enterprise-authenticated client not available for '<enterprise>'.
This happens even when no app_installations config exists at any level. Settings are still applied, but the check run concludes failure.
Expected: skip the plugin when no app_installations is configured at the org, suborg, or repo level, even when repos changed.
Workaround: add disable_plugins: [app_installations] to deployment-settings.yml.
- 主要语言
- JavaScript
- 星标
- 924
- 派生
- 227
- 平均合并
- 6 小时 8 分钟
- 30 天内合并 PR
- 15
环境准备
在浏览器里用你自己的 GitHub 账号启动这个项目的开发容器。
- 提供 Dockerfile 或 Docker Compose 文件
- 没有 Pull Request 模板
- 阅读贡献指南
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
github-community-projects/safe-settings 的其他 Issue
-
Repos transferred into the org are not synced (no repository.transferred handler)可能已有人在做 @breymander 于 3 天前认领。 未关闭
难度 2/5 1-3 小时 新手友好度 70/100
github-community-projects/safe-settings#1107 · 1 条评论 ·
维护者通常 1 天内回复
-
Dry run only inspects the first 30 changed files (pulls.listFiles is not paginated)可能已有人在做 @3rwww1 于 25 天前认领。 未关闭
难度 2/5 1-3 小时 新手友好度 84/100
github-community-projects/safe-settings#1080 ·
维护者通常 1 天内回复
-
PR comment diff table shows an empty vertical scrollbar due to malformed <tr> markup可能重新可做 关联的 PR 已关闭且未合并。 未关闭
难度 1/5 1 小时以内 新手友好度 88/100
github-community-projects/safe-settings#1077 ·
维护者通常 1 天内回复
-
Bug Report: environment and repo variables API calls don't paginate, failing with more than 10 items可能已有人在做 @cdav 于 78 天前认领。 未关闭
难度 2/5 1-3 小时 新手友好度 85/100
github-community-projects/safe-settings#1040 ·
维护者通常 1 天内回复
-
Teams plugin: PR comment/nop diff shows phantom additions+deletions for unchanged teams due to name vs. slug mismatch可能已有人在做 @StephHope 于 56 天前认领。 未关闭bug
难度 2/5 1-3 小时 新手友好度 76/100
github-community-projects/safe-settings#1033 ·
维护者通常 1 天内回复
查看 github-community-projects/safe-settings 的全部 Issue
相似的 Issue
-
data
难度 1/5 1 小时以内 新手友好度 74/100
gaseous-project/hasheous#637 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 62/100
gramps-project/gramps-web#1549 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 78/100
-
change request evaluating
难度 2/5 1-3 小时 新手友好度 72/100
sindresorhus/eslint-plugin-unicorn#3907 ·
维护者通常 1 天内回复
-
Priority: Low Production Type: Bug
难度 2/5 1-3 小时 新手友好度 68/100
GrottoCenter/grottocenter-api#1903 ·
维护者通常 1 天内回复