コミット時にclang formatでc,hファイルをコンテナ内でフォーマット

[nakamo326]

変更内容

• コミット時にbackendコンテナでステージングされてるc,hファイルをフォーマット
• makeの依存関係に.git/hooks内のスクリプトを追加

なぜ変更したか

• macとlinuxでclang-formatのバージョンが合わないことがままあるのでフォーマットの環境をコンテナに統一したい

動作確認の方法、動作確認した内容

• make後、c,hファイルに不要なインデントなどを追加してコミットを試す

その他、レビューで確認してほしいことなど

[s-xix98]

こんな感じでコピーしないと動かないの面倒だね。

自分の環境だと docker compose image の 名前が、de-backend なんだけど、
ナカモトさんの環境だと、de_backendになってるのか？

あと、自分の環境だと上記が原因でうまく動かなかったんだけど、
コミットには成功しちゃったから、
シェルスクリプトの一番最初に set -eux (x は なくてもいいかも)
とかつけるといいかも。

あと、docker compose が 動作中だと clang-format の コンテナが exit した時に、よくわからんコンテナ名で exit 0 とか出るから、
run の オプションの時に、 --name de-backend-clang-format とかつけてあげてほしいかも。

[nakamo326]

あら
docker-composeが作るイメージの名前環境で違うのかー
確かにイメージ別に用意するほうがよさそうだね
修正します

[nakamo326]

• フォーマット用のイメージ追加ので環境依存ではなくなったはず
• set -euつけたからdocker run失敗したら落ちるはず
• 他、細かい出力修正など

[nakamo326]

これフォーマットでdiffなくなると空コミットできちゃうな
ちょっと考えるます

[nakamo326]

ready for review~~

[s-xix98]

これは、どこから、どこに コピーしてんだ？

[nakamo326]

dockerのマルチステージビルドみたいなやつ
1行目のbuildって名前つけたやつから、6行目から始まるステージにコピーしてる

[nakamo326]

一旦保留