每次 commit 之前再自己 review 一遍代码是个好习惯

[Livid]

最近一年多来养成的一个习惯，就是每次 commit 之前，一定会在 GitX 里把要 stage 的更改再看一遍，而且 GitX 有一个好功能就是，你可以不用 stage 整个文件，你可以只 stage 某个函数中更改了的一个部分。

如果你还在用 git commit -a 的话，或许你可以考虑一下让自己的 commit 冲动慢一点，这样对代码质量会有好处。

[kaichen]

如果熟悉在Command line使用Git，不要使用`-m`，而是在Commit的时候加上`-v`，这样在编辑Commit Message时先往下走，把代码再认真看一遍，看好了再写Commit Message。

这是我一直以来的习惯。

[bitbegin]

GitX是啥？

[Livid]

@bitbegin /t/11773

[isy]

用 Vim 的话配合 vim-fugitive 插件也挺好用的， :Gdiff 对比很方便。 另外 git commit -a 是个很坏的习惯

[yangruixuan]

我是处女座我骄傲

[TankyWoo]

@kaichen 赞成，`-m`最好不要加，除非是简短的说明；还有就是没有特殊理由，不要使用`git add *`，`git add .`，也不要直接用`git commit -a`等。

主要还是最好再次确认下，防止比如把一些不需要的如隐藏文件等也给add提交进去了。

[yangg]

@kaichen Tks, I used to use `git dc' which is aliased to `diff --cached'

[ispinfx]

Magit帮到你

[leofml]

所以我一般用 SourceTree

[h2ero]

一般都是用的git diff --cached看下， 看来以后要养成git ci -v的习惯。

[yangqi]

这就是用GUI的好处,每次commit之前都可以很自然的review, 也便于更好的写comment

[powerfj]

提交之前检查代码是非常必要的，经常看别人一不小心带出去不该带出去的代码的..每次看别人烟都不眨一下全量提交我就在为他捏把汗..

[Phariel]

sourcetree一样，神器。

[ChiangDi]

JB家 的 IDE 都是 commit 的时候可以看的，关键是看到不对还可以直接改，简直神奇

[xujialiang]

虽然公司还在用svn。。。。。。不过我commit之前还是都看一遍。。。。。。。。xcode逼着我看啊~~~~~~~~~~~~~~~不看不舒服。。。。

[akfish]

Commit前review是必须的，我个人经常用pre-commit hook执行build和test，测试不通过直接就不能commit。

[passluo]

所以 SourceTree 的交互蛮好的，鼓励大家 commit 前先浏览一下自己的更改。

[yxjxx]

我觉得sourcetree比gitx更好用些,也有stage lines功能,还可以查看uncommitted changes,功能更丰富些.

[yangqi]

sourcetree +1, 同时支持git和mercurial

[lyragosa]

看来v2ex只有我一个人直接在线上改服务器代码QAQ