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

[Livid]

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

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

[jianzong]

一般直接git diff确认一下，刚刚才发现-v 是个好东西

[mcfog]

作为一个有追求，负责任的开发，这是必须的，要把commit当成是一件自豪的事情

sign your work. http://pragmatictips.com/70

[hustlzp]

在用sourcetree看。

[gancl]

sourcetree +1

[SoloCompany]

直接在 JetBrains 里面提交代码就不会忘记自己 Review 了

[rainysia]

发现没linux下用户吗? 给抛弃了.

目前review用的qgit, 有时也用gitg 和Giggle.

[AstroProfundis]

@lyragosa 你不是一个人...

但最近我也改成先提交到仓库再到服务器上去 git pull 了...

[jprovim]

In zsh, use gca instead.

or type it yourself `git commit -v -a`

[amoblin]

我是这样做的：
1. git stash
2. git pull
3. git stash pop（这里可能要处理冲突）
4. git status
5. git add -u
6. git diff --cached
7. git commit（这里我设置了2个git hooks：pre-commit和prepare-commit-msg。前者会做一些自动更新文件的工作，后者会自动添加一些信息到commit里去）
8. git log 确认一下
9. 没问题就 git push

附上2个git hooks的链接： https://github.com/amoblin/MyDotFiles/tree/master/git-hooks

[softempire]

commit 之前自己review是我们强制要求的

[niko]

都会review，嗯~

[fsw90628]

看了 WWDC，苹果的团队会在 commit 之前运行一遍 performance 和 correctness 的测试。

[railgun]

不review就敢提交？哪来的勇气？

[wy315700]

git add 之前用 git diff查看

[wecoders]

SourceTree+1
好用，不贵

[duzhe0]

一直都是先svn diff 再svn ci 的

[ijse]

一般用git diff

windows下内置了一个小工具 gitk, 还可以看提交树， 很不错~~　

另外，可以通过配置git hook实现一些commit之前要做的事，比如跑单元测试，检查代码格式 什么的。。

[bigporker]

这个不是标配么

每次commit之前，自己review一遍代码，再让同事帮忙review一遍代码是个好习惯

[yangqi]

@bigporker 自己review就够了, 同事review应该是commit之后干的吧

[ceyes]

commit 只要没有push上去就可以随便修改, rebase, reset……
所以我更倾向于在push 之前review 这个commit, 或发patch 让大家review。