1. quattro_4

    Posted

    quattro_4
Changes in title
+gitコミット時にデバッグコードが含まれていないかをチェックする
Changes in tags
Changes in body
Source | HTML | Preview
@@ -0,0 +1,58 @@
+すべてのgitユーザーにおすすめのhook
+
+`debugger`や`console.log`などコミットするべきではないものをコミットすることはありませんか?
+もしくは他のメンバーがコミットしていてイラッとしたことはありませんか?
+
+そんな時はgitのhook(pre-commit)を使います。
+
+以下はこのhookを使うための準備です。
+[gitのhooksを管理する(自分用テンプレートを使う)](http://qiita.com/items/59fdf8b9aa9ef48ecbdf)
+[git hookでできること](http://qiita.com/items/d8a8d498c9f853a445f5)
+
+pre-commitのコードは以下の通り(ruby)
+
+**`checkes`の内容を変えて、自分のチェックしたいコードを追加できます。**
+
+<script src="https://gist.github.com/4362089.js"></script>
+
+---
+
+使うにはとにかく各プロジェクトの`.git/hooks/`配下に`pre-commit`を配置し、実行権限を与えれば動きます。
+(権限重要で実行権限が無いと、何も起きない)
+
+```bash:
+$ cd rails_project
+$ cp /somewhere/pre-commit .git/hooks/
+(OR $ vi .git/hooks/pre-commit)
+$ chmod 755 .git/hooks/pre-commit
+```
+
+デバッグコードを含むコードをコミットしようとすると
+
+```bash:
+$ git commit
+'debugger' found in file: app/controllers/posts_controller.rb
+ -> + debugger
+'puts' found in file: app/controllers/posts_controller.rb
+ -> + #puts params.inspect
+COMMIT REJECTED. Please remove them before commiting OR use --no-verify
+```
+
+のように出力されます。
+
+**一時的にチェックを無効にしてコミットしたい場合は**
+
+```bash:
+$ git commit -m "Commit without checks" --no-verify
+```
+
+--------------------------------
+
+git hooksはプロジェクトごと.git/hooksにありますが、既存の数あるプロジェクトで有効にするには少し手間がかかります。
+詳細は[gitのhooksを管理する(自分用テンプレートを使う)](http://qiita.com/items/59fdf8b9aa9ef48ecbdf) 参照
+
+---
+
+デバッグコードはproduction環境であっても比較的無害な気はしますが、一度Javascriptのコードがconsole.logのせいで動かなかったことがあります。(特定のレガシーブラウザで起こった)
+
+個人的にはデバッグコードをあまりに頻繁にコミットしてプッシュまでしてしまう人は、不注意だったりコードの読みやすさ・コミットの作り方の意識が低いのかなと思う。(それだけバグも作りやすい人)