nodejs / nodejs/node-core-utils

Allow `request-ci` on the collaborator's PR without restriction of at least 1 approval

オープン
#937 コメント 6 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

主要言語
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?

コントリビューションガイド

コントリビューションガイドを開く

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. 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

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。