nodejs / nodejs/node-core-utils
Allow `request-ci` on the collaborator's PR without restriction of at least 1 approval
まだ誰も着手していません。
- 主要言語
- JavaScript
- スター
- 313
- フォーク
- 137
- 平均マージ
- 2日 8時間
- マージ済み PR(30日)
- 24
説明
IIRC collaborators are able to manually trigger CI through Jenkins without needing 1 approval on the PR, maybe it makes sense for the collaborators to use request-ci label on their own or other collaborators's PR without at least 1 approval restriction.
https://github.com/nodejs/node-core-utils/blob/a68af4b7744e8416bfc0f062570688b67d5e10dc/lib/pr_checker.js#L560 just an idea - could we check either the PR author is a current collaborator?
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
調査の方向性
lib/pr_checker.js の560行目付近から始め、request-ci ラベルで使用されている承認制限を追跡します。collaborator のステータスと pull request の作成者情報がどのように表現されているかを確認し、関連する CI ラベルのチェックを検証します。完了条件は、対象となる collaborator が、自分自身または別の collaborator の pull request に対して、1件の承認を必要とせず request-ci を使用できることです。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- javascript
- 領域
- authorization, ci-cd
- issue の種類
- 機能追加
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 活発さ
- 静か
- 明瞭さ
- おおむね明確
- 初心者へのやさしさ
- 56/100