search

kirikirisu / kanazawaApp-2019

stockSupporter
0 stars 1 forks source link

app/.gitignoreにbuild.gradleを追加 #35

Closed tomu28 closed 4 years ago

tomu28 commented 4 years ago

Issueの番号 / Title

close #33

どのように解決したか / How

.gitignoreにbuild.gradleを追加

なぜそのような解決策を取ったのか / Why

.gitignoreになかったため

daichi77 commented 4 years ago

これmergeしていんじゃない？

kirikirisu commented 4 years ago

よくわからなかったため調べてみましたが、かなっこや 適当なサンプルのignoreにbuild.gradleはないみたいです。 buildする際の環境は全員で揃える必要はないということですか？

daichi77 commented 4 years ago

ごめんmergeしていいと言ったものの、確かにignoreせずにチームで統一した方がいんじゃないかと思ってしまった。 これってなんでignoreに追加することになったんかな？

tomu28 commented 4 years ago

チームで統一した方がいいというのは賛成で、そのために現状の設定で問題なかったら、ignoreした方が開発が楽かなって思って提案しました。 前回、他の人が作業しているときにこのファイルの差分が生まれていたから追記しておいた方がいいかなって考えていました。

ちなみに、 ルートの build.gradle ではなく、 app/build.gradle のignoreです。

daichi77 commented 4 years ago

例えばだけどConstraintLayout使うときってbuild.gradleいじらないと使えなくて、そういうケースってもしかしたら今後もあると思うんですけど、そういう時の差分は各々やってもらうって感じでしょうか

tomu28 commented 4 years ago

あぁ〜なるほど、その場合は変更commitしないとダメか

ちょっと把握し切れていない部分になるんだけど、その場合ってルートの build.gradle に変更が生じるんかな

もし、app/build.gradle の場合はignoreしたら良くないってことになるね

daichi77 commented 4 years ago

https://riptutorial.com/ja/android-gradle/example/7075/android-studio%E3%83%97%E3%83%AD%E3%82%B8%E3%82%A7%E3%82%AF%E3%83%88%E3%81%ABbuild-gradle%E3%83%95%E3%82%A1%E3%82%A4%E3%83%AB%E3%81%8C2%E3%81%A4%E3%81%82%E3%82%8B%E3%81%AE%E3%81%AF%E3%81%AA%E3%81%9C%E3%81%A7%E3%81%99%E3%81%8B- rootの方はignoreでいいのかなと思ったりしてます。appの方はしない方がいい気がする

tomu28 commented 4 years ago 
## モジュールレベルのビルドファイル
これらのビルド設定を構成すると、追加のビルドタイプと製品のフレーバーとして使用し、 main/ appマニフェストまたは最上位のbuild.gradleファイルの設定を上書きします。

とあるから、rootの方のbuild.gradleはignoreして、 appの方はそのままにしておいた方が良いね

tomu28 commented 4 years ago

上記コメントの理由より、このPRはCloseします。