web-dev-qa-db-ja.com

修正する必要がある場合、TFSで[拒否]または[作成者を待つ]を使用する必要がありますか?

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

enter image description here

12
sashoalm

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

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

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

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

したがって、私はWaiting for Authorこれは、作成者がソリューションへのアプローチを台無しにしたと考えているが、コメントを心に留めておけば、コードは引き換え可能であることを意味します。

そしてRejectedは、コードがどれほど上手く書かれているかにかかわらず、このような変更を受け入れられないことを意味します。

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

14
Peter M