web-dev-qa-db-ja.com

Gitlabを使用してコードレビューを設定する方法

Gitlabを使用してコードレビューを設定するにはどうすればよいですか？ GitlabのWebサイトに機能としてリストされていますが、設定方法についての説明は見つかりません（そのため、Gitlabユーザーマニュアルへのリンクをお勧めします）。

私の検索のいくつかは、「マージ要求」が進むべき方法であることを示しています...しかし、私はそれらを制限していると感じています。発行されたマージ要求には、1つのブランチと他のブランチ間のすべてのコミットが表示されます。私は、個々のコミットごとに生成された差分のみを表示できるようです。たとえば、レビューしたいファイルがあるとします。これは新しいファイルですが、devブランチで10を超えるコミットで変更を送信しました。統合からそのdevブランチに対してマージ要求を発行すると、ファイルに加えられた増分変更を示す10個のコミットが表示されます...全体を確認したいと思います。それは新しい！

ここで間違った木をbarえていますか？ GitLabで使用できる実際のコードレビューツールはありますか、それともマージリクエストをそのまま使用しますか？ここで適切なコードレビューを設定する最良の方法は何ですか？

gitgitlabpull-request
83
djc6535

注：GitLab 6.4以降では、 side-by-side diff view を使用できます。「 pull request 5308 "」を参照してください。

（2013年7月）ただし、ファイルレベルでのみ、各行にコメントする可能性はまだありません。
Daniel Sokolowski 言及 コメント内 that行ごとのコメントがサポートされるようになりました（2014年9月）：

チームメンバーは、マージリクエストに一般的にコメントするか、行コメントを使用して特定の行にコメントすることができます。

それでもコードレビューアクティビティに役立ちます。

https://f.cloud.github.com/assets/4224518/1558702/e0fe633a-4fa3-11e3-9388-3f3e445cb6d4.png

23
VonC

Gitlabで2か月以上、ほとんど摩擦なくコードレビューを行ってきました。開発者が新しいコミットをプッシュするたびに電子メール通知を送信するように rss2email を設定しました。次に、コミットにGitlabのコメント機能を使用して、プッシュされたコードに関するコメントを作成します。

残念ながら、Gitlabはファイル自体へのコメントを許可せず、コミットのみで許可します（Githubのように）。前回のコミットで見逃したことをコメントする必要がある状況にいるときはいつでも、コメントツールを使用して、コメントするコードセクションを導入/変更したコミットを見つけます。

完璧とはほど遠いですが、今のところうまく機能しています。

8
Herberth Amaral

送信されたコードは、他のリポジトリのマージリクエストまたは現在のリポジトリで確認できます。
example http://demo.gitlab.com/diaspora/diaspora/commits/master

その後、コミットされたファイルの変更にコメントを追加できます（ボタン Reply）またはコミット全体

例 http://demo.gitlab.com/diaspora/diaspora/commit/42f47626890218a180870bc3f44ec57625b0779c

結果の 通信はコードレビュー 。ただし、個人的には、可能な場合は1対1の通信で1台のPCでコードレビューを行い、結果を記録するツールや、より形式的なものが必要な場合にツールを使用することをお勧めします。

多くのコミットがあるファイルレビューの場合、例えば http://demo.gitlab.com/diaspora/diaspora/blame/master/README.md を使用して見て blame 誰が何をしたかを理解します。ただし、このビューには、通信してコメントを追加するオプションはありません。この場合、コメントとして変更を追加することをお勧めします。

3
Paul Verest

はい。マージリクエストは、ピアレビューの達成方法です。

すべてのコミットの変更を表示する「差分」タブがあるはずです（ここで説明します： http://youtu.be/DyAX8ws5OIc?t=3m2s ）。

ビデオでは、ピアレビューにどのように使用できるかもわかりやすく説明しています。

0
onionjake

コードレビューの通常のユースケースは、マスターなどにマージする前にブランチのコードをレビューすることです。プロジェクトを開発し、すべてのコードをチームの全員がレビューしたい状況があります。

私がしたことは：

最初のコミットをチェックアウトし、変更を加え、コミットしてプッシュします

git co -b FIRST_COMMIT eb67f06c2b3222c0219214b176c41922bc454881
vi README.md
git add README.md
git ci -m "First commit modified so can get full diff against it"
git Push --set-upstream Origin FIRST-COMMIT

最後のコミットをチェックアウトし、変更を加え、コミットしてプッシュします

git co -b master
vi README.md
git add README.md
git ci -m "Last commit modified so can get full diff against it"
git Push --set-upstream Origin LAST-COMMIT

GitLab/GitHubで、プルリクエストを作成します

  • LAST_COMMITからFIRST_COMMITへのマージです

私のために働く！

0
HankCa