代码质量和 Pre commit

一个人写代码的时候，想怎么写就怎么写，能实现功能就非常不错了，何必去和代码质量计较呢，谈什么代码重构！但是在 teamwork 中，这是一个不得不重视的问题。

Paste_Image.png

可能你的 team中有代码 review，这个确认能解决不少的问题，但是人眼在某些时候会远不如机器，如果你在 review 别人代码的时，发现有些地方空行太多，命名格式不规范等基本代码功力问题，必定会火冒三丈，心想：我 tm 百忙之中，抽空reveiw 你的代码，结果基本规范都没有通过，简直只浪费我的时间。

低级问题不应该提交到远程仓库，所有提交代码前的审查变得非常有必要。Git，Svn 等版本控制工具支持很多 hooks。以 git hooks 为例 , 钩子(hooks)是一些在"$GIT-DIR/hooks"目录的脚本, 在被特定的事件(certain points)触发后被调用。

pre-commit

在commit 之前会被触发，编程语言以 javascript 为例，如果能在 js 文件提交之前，执行一次 jshint ，那就满足我们的需求了。可最为一个追求极致的人怎么能只满足呢。

最优的模式，

1. jshint 检查问题
2. beauty code
3. format code
4. auto review
   ...

这些都是在本地完成的，当然你也可以用服务器端 check 的方式。比如 code climate、travis、jinkens 等工具。

今天的主角是 http://pre-commit.com/

安装

Before you can run hooks, you need to have the pre-commit package manager installed.

Using pip:

pip install pre-commit

Non Administrative Installation:

curl http://pre-commit.com/install-local.py | python

System Level Install:

curl https://bootstrap.pypa.io/get-pip.py | sudo python - pre-commit

In a Python Project, add the following to your requirements.txt (or requirements-dev.txt):

pre-commit

使用

1. 进入你的项目目录
2. 执行 pre-commit install
3. 新建文件 .pre-commit-config.yaml。yaml 的语法参考
4. 编辑你的配置文件

example

-   repo: git://github.com/pre-commit/mirrors-jshint
    sha: 'v2.8.0'  # Use the sha you want to point at
    hooks:
    -   id: jshint
        name: jshint
        entry: jshint
        language: node
        files: \.js$
        args: []
        #excludes:

        # Use additional_dependencies to install the actual node package
        additional_dependencies: ['jshint@2.8.0']
        minimum_pre_commit_version: 0.7.0

更多 hooks

pre-commit 支持非常多的 hooks，配置方法也很简单，只需要你修改 .pre-commit-config.yaml 文件即可，更多 hooks 参考以下截图

http://pre-commit.com/hooks.html

Paste_Image.png