Allow `request-ci` on the collaborator's PR without restriction of at least 1 approval
まだ誰も着手していません。
評価
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 初心者へのやさしさ
- 56/100
- issue の種類
- 機能追加
- 明瞭さ
- おおむね明確
- 活発さ
- 静か
- 技術スタック
- javascript
- 領域
- authorization, ci-cd
調査の方向性
lib/pr_checker.js の560行目付近から始め、request-ci ラベルで使用されている承認制限を追跡します。collaborator のステータスと pull request の作成者情報がどのように表現されているかを確認し、関連する CI ラベルのチェックを検証します。完了条件は、対象となる collaborator が、自分自身または別の collaborator の pull request に対して、1件の承認を必要とせず request-ci を使用できることです。
索引モデルが issue の本文から書いたものです。
説明
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?
- 主要言語
- JavaScript
- スター
- 313
- フォーク
- 137
- 平均マージ
- 2日 8時間
- マージ済み PR(30日)
- 24
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
nodejs/node-core-utils のほかの issue
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
nodejs/node-core-utils#664 · コメント 1 件 ·
-
難易度 3/5 1〜2日 初心者へのやさしさ 65/100
nodejs/node-core-utils#1164 ·
-
難易度 4/5 3〜5日 初心者へのやさしさ 68/100
nodejs/node-core-utils#1160 · コメント 3 件 ·
-
難易度 3/5 1〜2日 初心者へのやさしさ 68/100
nodejs/node-core-utils#1151 ·
-
難易度 5/5 1週間以上 初心者へのやさしさ 35/100
nodejs/node-core-utils#1063 ·
nodejs/node-core-utils の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
palladius/rails8-app-on-gcp#145 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
dotenvx/dotenv-vscode#139 ·
-
test-change-proposal
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
web-platform-tests/interop#1455 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
corsairdev/corsair#1764 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100