

GitHubでPull
requestを作成して他のメンバーにコードレビューしてもらう際、静的なファイルや自動生成されたファイルを表示させないようにする方法はないでしょうか？

外部の有料CSSテンプレートなどを導入すると、自分で書いたわけではないCSS、JS、画像等が大量にpull requestに含まれる場合があります。  
こうしたファイルは特別な理由がない限りコードレビューしてもらう必要はありません。  
それよりも、自分自身が書いたコードを重点的にレビューしてもらうことの方が重要です。

しかし、普通にpull requestを作成すると、自分が書いたコードも外部から持ってきたファイルもすべてpull
requestのdiffに含まれるため、第三者が見たときに非常に効率が悪くなります。  
また、数が多すぎるとGitHubが白旗を揚げる場合もあります。（下記イメージ参照）

![GitHubが白旗を揚げる例](https://i.stack.imgur.com/DbAFw.png)

こうした問題を避けるために、特定のディレクトリ以下は（diff上では）無視するとか、別々のpull
requestにするとか、何かしらの回避策があれば教えていただきたいです。

よろしくお願いします。


CC BY-SA 3.0
2015-02-11T11:10:11.543
6317
2020-10-03T06:15:41.070
85
question
git github
Pull request時に静的なファイルや自動生成されたファイルをGitHub上のdiffに表示しないようにするには？


多少煩雑ではありますが、Pull requestのコミットを

  1. 静的ファイル、自動生成ファイルの追加
  2. レビューして欲しい変更

の複数段階に分けて、レビューアにはPull requestの番号と2.のコミットへのリンクを提示する、という方法ではいかがでしょうか。  
（というようなblogかQiitaの記事を見た覚えがあるのですが、所在不明です……）


CC BY-SA 3.0
2015-02-12T03:40:47.157
6355
2015-02-12T03:40:47.157
2590
6317
answer


差分を出したくない静的ファイルのみを commit した branch (A) とレビューしたいコードを commit した branch (B)
を分け、GitHub 上で PullRequest を作る時に、base を A、compare を B に指定していました。

具体的な手順としては、以下のような感じです。  
git-flow モデルで運用している前提とします。

  1. git checkout develop
  2. git checkout -b feature/A
  3. git add staticfiles && git commit -m 'commit static files'
  4. git push origin feature/A
  5. git checkout -b feature/B
  6. git add pleasereview && git commit -m 'please review this commit'
  7. git push origin feature/B
  8. GitHub 上で以下のように指定して PullRequest を作る。  
![画像の説明をここに入力](https://i.stack.imgur.com/pL2Q8.png)

develop へのマージ時は以下の点に、気をつけなければなりません。  
必ず、feature/B を feature/A にマージした後、feature/A を develop にマージするということです。

diff が大量に出て GitHub 上でのレビュー運用がうまくいかないという点を解消するには、個人的には今のところベストプラクティスかなと考えています。


CC BY-SA 3.0
2015-02-12T04:15:07.093
6357
2015-02-12T04:15:07.093
3222
6317
answer


ジャストアイデアで実際には試していませんが、以下のような手順だとうまくいくかもしれません。

  1. 一通りコードが出来上がった時点で静的ファイルが含まれるディレクトリを削除してコミット
  2. Pull request作成、コードレビュー依頼
  3. 必要に応じてコードを修正、コミット、push
  4. コードレビューが終わったら、1のコミットをrevert＆GitHubにpush
  5. Pull requestをマージ

懸念等があればコメントをください。


CC BY-SA 3.0
2015-02-12T04:36:07.457
6359
2015-02-12T04:36:07.457
85
6317
answer


なぜ日本語ではgithub linguistの情報が少ないのか

というわけで記事書きました。おそらく質問者さんがやりたいことが出来るかと思います。

[GithubのPullRequestで、自動生成されたファイルなどのdiffを非表示にする -
Qiita](https://qiita.com/takezoux2/items/00a60d524cc0e787e6c3)


CC BY-SA 4.0
2020-10-02T08:59:27.960
70885
2020-10-03T06:15:41.070
2020-10-03T06:15:41.070
3060
7693
6317
answer