レビュー
Library Team、Library API Team、および Library Contributors のすべてのメンバーには「r+ 権限」があります。
つまり、PR を承認し、@bors に対して
それをテストして Rust nightly にマージするよう指示する能力です。
PR をレビューすることにしたなら、ありがとうございます! ただし、以下の点に留意してください。
- 誰に割り当てられているかにかかわらず、どの PR でもレビューすることはいつでも歓迎されます。
ただし、以下の場合を除き、PR を承認しないでください。
- 他の誰も先にレビューしたいと思っていないと確信できる場合。チームの他の誰かの方がレビュー担当者として適任だと思う場合は、遠慮なくその人に再割り当てしてください。
- コードのその部分に自信がある場合。
- 破損やパフォーマンスの回帰を引き起こさないと確信できる場合。
- 変更について完了した FCP がある場合を除き、ドキュメントで行っている安定性に関する約束を含め、公開 API を変更しない場合。
- 不安定 API の変更/追加については、設計が小さく、変更に議論の余地がない場合、RFC プロセスを省略することが許容される場合があります。
そのような変更には必ず
@rust-lang/libs-apiを関与させてください。
- 不安定 API の変更/追加については、設計が小さく、変更に議論の余地がない場合、RFC プロセスを省略することが許容される場合があります。
そのような変更には必ず
- レビュー時は常に礼儀正しくしてください。あなたは Rust プロジェクトの代表者であるため、行動規範に関しては期待以上の対応をすることが期待されています。
レビューの詳細については、https://forge.rust-lang.org/compiler/reviews.html を参照してください。
High-five ローテーション
チームの一部のメンバーは「high-five ローテーション」に参加しています。 これは、high-five bot が新しい PR の割り当て先レビュアーを選ぶリストです。
いずれかのチームのメンバーであることは、このリストに載ることを期待されるものではありません。 ただし、このリストのメンバーは、3 つのライブラリチームの少なくとも 1 つに所属している必要があります。
bot があなたに PR を割り当てたものの、それをレビューする時間や専門知識がない場合は、
遠慮なく他の誰かに再割り当てしてください。
high-five ローテーションから選ばれた別のランダムな人に割り当てるには、
r? rust-lang/libs を使用してください。
長期間にわたってレビューを行えない状況になった場合は、 リストから自分を(一時的に)削除するのがよいかもしれません。 リストに自分を追加または削除するには、 triagebot 設定ファイルを変更する PR を送ってください。
ロールアップ
ライブラリ PR では、ロールアップ(@bors r+ rollup)は多くの場合問題ありません。
特に、新しい不安定な追加のみの場合や、ドキュメントのみに触れる場合です。
パフォーマンスに影響する PR はロールアップすべきではありません(@bors rollup=never)。
微妙なプラットフォーム固有の変更を含む PR も、ロールアップの候補として適していない場合があります。
ロールアップすべきタイミングの詳細については、
ロールアップのガイドラインを参照してください。