app_installations reports an error on every repo or suborg change when the plugin is not configured
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 初心者へのやさしさ
- 64/100
- issue の種類
- バグ
- 明瞭さ
- 明確に書かれている
- 活発さ
- 活発
- 技術スタック
- javascript
- 領域
- backend
調査の方向性
lib/settings.js から始めてください。hasChangedConfigs が真のとき syncSelectedRepos が syncAppInstallations を呼び出しており、Enterprise 認証済みクライアントが必要になる箇所を見つけてください。組織、サブ組織、リポジトリのいずれのレベルにも app_installations の設定が存在しない場合、リポジトリに変更があってもプラグインが早期に return するようガードを追加してください。Enterprise のインストールも 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分
- マージ済み PR（30日）
- 15
環境構築
このプロジェクトの開発コンテナを、あなたの GitHub アカウントでブラウザ上に起動します。
- Dockerfile または Docker Compose ファイルあり
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
github-community-projects/safe-settings のほかの issue
-
Repos transferred into the org are not synced (no repository.transferred handler)対応中かも @breymander が 5 日前に担当しました。 オープン
難易度 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 が 28 日前に担当しました。 オープン
難易度 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再び着手できるかも このイシューのプルリクエストはマージされずにクローズされました。 オープン
難易度 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 が 80 日前に担当しました。 オープン
難易度 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 が 59 日前に担当しました。 オープンbug
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
github-community-projects/safe-settings#1033 ·
メンテナーはふだん 1 日以内に返信
github-community-projects/safe-settings の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
メンテナーはふだん 1 日以内に返信
-
maintenance
難易度 1/5 1時間未満 初心者へのやさしさ 75/100
alphagov/accessible-autocomplete#826 ·
メンテナーはふだん 4 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 77/100
olahol/react-tagsinput#229 ·
-
[BUG] npm pkg delete removes the first array item for a nonexistent property対応中かも このイシューにリンクされたプルリクエストがオープン中、またはマージ済みです。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 83/100
メンテナーはふだん 2 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 74/100
jonathanong/no-mistakes#2260 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信