修正する必要がある場合、TFSで著者の拒否または待機を使用する必要がありますか?


12

TFSでは、プルリクエストで修正する内容についてコメントを書き込んでから、それを受け入れる場合、それを拒否または作成者としてマークする必要がありますか?どちらが良いですか?

ここに画像の説明を入力してください

回答:


14

プルリクエスト付きのレビューコードの Microsoftのとおり:変更に投票して、承認の各クラスの推奨される目的は次のとおりです。

提案を承認:プルリクエストに同意しますが、コードを改善するためのオプションの提案を提供します。

作成者を待機しています:変更を承認せず、作成者にコメントの確認を依頼してください。作成者は、懸念事項に対処した後、いつコードを再レビューする必要があるかを通知する必要があります。

拒否:変更は受け入れられません。この方法で投票する場合は、プルリクエストにコメントを残して、変更が拒否された理由を詳しく説明する必要があります。

したがって、私Waiting for Authorはあなたが作者が彼/彼女の解決策へのアプローチを台無しにしたと思うが、彼/彼女のコードがあなたのコメントを心に留めるなら引き換え可能であることを意味すると思います。

そしてRejected、コードがどれほど上手く書かれていようとも、このような変更を受け入れることはできません。

あなたが自問しなければならない問題は、提案された目的についてのグループの考えがMicrosoftの考えと一致するかどうかです。


1
これについて:「懸念事項に対処した後、いつコードを再レビューする必要があるかを作成者から通知する必要があります。」-PRが再レビューの準備ができていることをレビューアに警告するための作成者のメカニズムはありますか?
ロバート・シム

@RobertSim-再レビューを求めるメッセージを送信するために、[レビュー担当者]の[Ping]オプションを使用します。
killercowuk 2018年
弊社のサイトを使用することにより、あなたは弊社のクッキーポリシーおよびプライバシーポリシーを読み、理解したものとみなされます。
Licensed under cc by-sa 3.0 with attribution required.