bitbucketでプルリクエストのコードをマージできる人を制限する方法は？

Question

私はgitリポジトリとしてbitbucketを使用する小さな開発チームを持っています。

私はbitbucketのプルリクエストでコードをマージできる人を制限する方法を知りたいですか？ AND/ORは、マージが実行される前に、少なくとも1つの承認を強制します。基本的に私はコードレビューを強制しようとしています。

現在のところ、プルリクエストの作成者（およびその他の人）は、品質目的で問題になるコードを承認するだけでなく、マージすることもできます。前もって感謝します。

更新：

のBitbucketは、今プッシュ権限、枝の削除、および履歴の再書き込みを制御することができます。 https://confluence.atlassian.com/bitbucket/branch-management-385912271.html

ただし、最小限の承認数を強制する方法はありません。 attlassianから

Answer 1

bitbucketのプルリクエストでコードをマージできる人を制限する方法を知りたいのですが？ AND/ORは、マージが実行される前に、少なくとも1つの承認を強制します。基本的に私はコードレビューを強制しようとしています。

この機能は現在Bitbucketでは利用できませんが、Atlassianのファイアウォールの背後にあるGitのホスティングにはこの機能があります。

スタッシュは、あなたがすることができます：プルリクエストをマージする前に承認の最小数を強制枝

を変更することができます

• が限界（それは竹のビルドのために同様のことを行うことができます - コードすなわちプルリクエストが

を変更した場合）、それをマージすることができます前に、

リセット承認をコンパイルする必要があります

アトラシアン自身の製品には、不思議なアシンメトリーがあります。

Answer 2

回答：

メアリーアンソニー[アトラシアンのテクニカルライター]

こんにちは、だから、

、リポジトリは枝を持つことができます。そのリポジトリ内では、 リポジトリと異なるブランチに対してアクセス権を設定することはできません。 開発者 がリポジトリをフォークし、そのリポジトリに対してプル要求を発行できるように、リポジトリに権限を設定できます。 設定するには、この：

Create a group on your account and call it "developers". 
Give the group read permissions. 
Add all the developers to that group. 
Edit the groups on the repository and add developers. 

は、この情報がお役に立てば幸いです。

メアリーここで

それは次のとおりです。https://confluence.atlassian.com/display/BITBUCKET/Work+with+pull+requests?focusedCommentId=321851850#comment-321851850

言い換えれば、あなたはあなたのdevのフォーク彼らのフォークからのプロジェクトや問題のプルリクエストすることができます。プロジェクトでは、プロジェクトを設定してパブリックフォークを禁止することができます。私は彼らがプロジェクトをフォークし、それが隠されると仮定します。つまり、プルリクエストを発行して自分のリポジトリを編集できるようになるということです。それはかなり厄介なように見えますが、うまくいくはずです。

github/bitbucketなどの権限を処理する良い方法があるとは思いません。本当に

編集

ない、それを強制するための解決策が、それでもかなり有効。プルリクエストの承認は非常にオプションなので、あなたがいなくても、本当にあなたが悩んでいるというわけではありません。私はシステムを強制しようとはしません。現実には、コードレビューが重要です。プルリクエストにより、コミットのセットを簡単に確認できます。

私のチームでは、プルリクエストを作成/承認するのに数ヶ月働いていました。私が働いたチームは、プルリクエストが時間の無駄であると判断し、私が出るまでコードレビューをした人はいなかったと思います。私が最後に聞いたことは、チームメイトが現在どのように動作しているのかわからないため、自分のコードをリファクタリングしているということです。

私が言っていることは、コードレビューを強制すべきではなく、あなたのチームはそれを非常に重要なものとして見なければならないということです。あなたのチームの各メンバーは一緒に働き、自分たちでお互いのレビューをコードしなければなりません。コードレビューを行うことにより、彼らは「醜い」と感じるコードを拒否する権利を持つか、あるいは別の方法で設計する必要があります。各メンバーは、他の開発者が何をしているのかを最新の状態にしておくことができ、病気、出産、死亡の場合に誰かの仕事を変えることはほとんどできません！

システムのプロセスを強制することは、マネージャの観点からは良いことです。しかし、私はオプションとして承認を受けることは悪くないと考えています。そして、マネージャーの仕事は、1人以下の承認された人でプルリクエストの結合されたプルリクエストをチェックすることです。プルリクエストを合併した人と承認した人を確認します。とにかくコードを見直す人を探しましょう。

一方、プルリクエストが永遠にぶら下がっており、誰もそれを見直していない場合。メイトにレビューを依頼するのはdevのタスクです。