前端项目中使用husky做预检查

具备基本工程素养的同学都会注重编码规范,而代码风格检查(Code Linting,简称 Lint)是保障代码规范一致性的重要手段。

使用 Lint 会有什么好处呢?在我看来至少具有如下 3 点:

  • 更少的 Bug
  • 更高的开发效率,Lint 很容易发现低级的、显而易见的错误
  • 更高的可读性
  • 很多时候我们lint的校验是放在持续集成阶段,大概流程如下:

代码提交 --> 跑 CI 发现问题(远程) --> 本地修复问题 --> 重新提交 --> 通过检查(远程)

但这样有一个问题,我们的 CI(持续集成) 往往不是仅仅只做 Lint工作,它还有会有很多其它的任务(如打包文件,静态资源上传 CDN 等),这样就导致特别的浪费时间,往往可能需要几分钟之后你才会发现问题,或者有的时候你根本就没有发现你的 CI 没有跑通过。

常见的流程:本地写好了代码,提交,开始跑 lint,发现不通过,本地修改代码,再提交,再等待 CI 的结果,若还有问题再重复之前的操作。

husky 为 git commit 增加钩子

在之前的工作中,我们尝试通过在 git 的 pre-receive 阶段嵌入一系列的 ci 流程处理代码以提供给开发者们 "just push" 的开发流程(当然这个想法是完完全全源自 heroku 的)。这个流程将原先的 "push -> wait for verify -> new correct commit -> repush" 的流程转变为 "push -> fail -> correct -> repush":如果没有在 "pre-receive" 阶段设置门禁的话,坏的提交会被同步到中心仓库后在进行检测;而设置门禁之后坏的 commit 会被拒绝在本地,本地只能将 ci 可以通过的代码提交到中心仓库。但是将所有东西都通过 push 验证很显然是慢了一些:这就像表单的前端验证和后端验证一样,虽然后端验证永远必不可少但是它增加了服务器的负担并且延长了反馈周期。

这时候 husky 就要派上用场了。husky 其实就是一个为 git 客户端增加 hook 的工具。将其安装到所在仓库的过程中它会自动在 .git/ 目录下增加相应的钩子实现在 pre-commit 阶段就执行一系列流程保证每一个 commit 的正确性。部分在 cd commit stage 执行的命令可以挪动到本地执行,比如 lint 检查、比如单元测试。当然,pre-commit 阶段执行的命令当然要保证其速度不要太慢,每次 commit 都等很久也不是什么好的体验。

在项目根目录下安装

yarn add --dev husky lint-staged

修改package.json文件

 "husky": {
   "hooks": {
     "pre-commit": "lint-staged",
   }
 },
 "lint-staged": {
   "*.{js,vue}": [
     "eslint --fix",
     "git add"
   ]
 },

如上配置,每次它只会在你本地 commit 之前,校验你提交的内容是否符合你本地配置的 eslint规则,如果符合规则,则会提交成功。如果不符合它会自动执行 eslint --fix 尝试帮你自动修复,如果修复成功则会帮你把修复好的代码提交,如果失败,则会提示你错误,让你修好这个错误之后才能允许你提交代码。

优雅的提交你的Git Commit Message

在项目根目录下安装

yarn add --dev @commitlint/cli @commitlint/config-conventional

在项目根目录下新建.commitlintrc.js

module.exports = {
 extends: ['@commitlint/config-conventional'],
 rules: {
   'subject-case': [0, 'never'],
   'type-enum': [
     2,
     'always',
     [
       'build', // 构建
       'ci', // ci
       'chore', // Other changes that don't modify src or test files. 改变构建流程、或者增加依赖库、工具等
       'docs', // Adds or alters documentation. 仅仅修改了文档,比如README, CHANGELOG, CONTRIBUTE等等
       'feat', // Adds a new feature. 新增feature
       'fix', // Solves a bug. 修复bug
       'perf', // Improves performance. 优化相关,比如提升性能、体验
       'refactor', // Rewrites code without feature, performance or bug changes. 代码重构,没有加新功能或者修复bug
       'revert', // Reverts a previous commit. 回滚到上一个版本
       'style', // Improves formatting, white-space. 仅仅修改了空格、格式缩进、逗号等等,不改变代码逻辑
       'test' // Adds or modifies tests. 测试用例,包括单元测试、集成测试等
     ]
   ]
 }
}
修改package.json文件
 "husky": {
   "hooks": {
     ...
     "commit-msg": "commitlint -E HUSKY_GIT_PARAMS"
   }
 },

这样,每次提交都会检测message信息,如果不符合要求,则提交不成功。

prettier 保证每个团队代码格式一致性

多少年来开发者在使用 tab 还是 space 的问题上真是花费了不少的时间,美剧硅谷里主角还因为 tab 和别人闹了一集,可见大家对代码格式化的重视程度-_-。记得我在上一个项目里看到有小哥把我的代码强行刷成他满意的格式的 commit 也非常不满。仅仅修改格式的 commit 是毫无必要的,它没有对软件本身的行为做任何的修改,而夹带了修改格式的 commit 更是令人抓狂的,给 review 的同学也带来了不小的负担(在一坨提交里仔仔细细看了半天发现神马也没变!!尼玛!!)。

golang 取了个巧,语言自带官方格式,你们终于不吵了吧。虽然会有时候看 golang 的格式化结果略微有点麻烦(就是 struct 对 json type 的制表符对齐的要求),但是也没有哪里是让人无法忍受的丑。如果其他的语言也以类似的方式制定一个官方格式是不是就会将此事平息下去呢。当然,我们可以在制定这个官方格式的时候吵架,只要官方格式不会三天两头的更新那在实际项目中为这种不必要的分歧导致浪费大把时间了。

在我看来 prettier 就是这么一个 "类官方格式" 了。不过目前它还只是支持 js 体系下的格式化,其他语言由于这样那样的问题还要再等等。

prettier

大家对有个公认的格式这件事还是非常认可的,项目出现一年,Github star 破 2.1w,并且像 facebook 这样的大公司已经在内部逐渐铺开使用了。

集成

最后,通过 husky 为 prettierpre-commit 加个钩子,这体验就更完美了:不论你家的格式是什么样子,只要你想提交,就必须格式化成 prettier 要求的样子,这样就没有那种因为格式变动出现的无聊的 diff 了。集成的流程基本是以下这个样子:

  1. 添加 prettier 依赖

    yarn add prettier --dev --exact
    
    
  2. 测试格式化是否工作

    yarn prettier -- --write src/index.js
    
    
  3. 在 commit 时执行 prettier

    yarn add pretty-quick husky --dev
    
    

    修改 package.json 添加 pre-commit 钩子

    {
      "scripts": {
        "precommit": "pretty-quick --staged"
      }
    }
    
    

其实官方文档也有,但是官方文档可耻的写错了...第二步命令少了 -- 的命令。

最后的最后,放一段 prettier 格式化的 react 代码,我还是对其默认的格式非常满意的。

class Badges extends React.Component {
  componentDidMount() {
    let { user, loadBadges } = this.props;
    loadBadges(user.username);
  }

  render() {
    let { badges } = this.props;
    return (
      <div style={{ marginTop: "50px" }}>
        <h1>已经获得的成就</h1>

        <Row
          gutter={16}
          type="flex"
          justify="center"
          align="top"
          style={{ marginTop: "20px", paddingBottom: "10px" }}
        >
          {badges.map(badge => (
            <Col
              lg={6}
              md={6}
              sm={8}
              xs={12}
              style={{ marginBottom: "1em" }}
              key={badge.project.id}
            >
              <ProjectBadge badge={badge} />
            </Col>
          ))}
        </Row>
      </div>
    );
  }
}

相关资料

  1. Prettier
  2. Husky
  3. Using Prettier and husky to make your commits safe.
  4. git-hooks
  5. The Commit Stage
  6. Formats Go programs
最后编辑于
©著作权归作者所有,转载或内容合作请联系作者
  • 序言:七十年代末,一起剥皮案震惊了整个滨河市,随后出现的几起案子,更是在滨河造成了极大的恐慌,老刑警刘岩,带你破解...
    沈念sama阅读 162,050评论 4 370
  • 序言:滨河连续发生了三起死亡事件,死亡现场离奇诡异,居然都是意外死亡,警方通过查阅死者的电脑和手机,发现死者居然都...
    沈念sama阅读 68,538评论 1 306
  • 文/潘晓璐 我一进店门,熙熙楼的掌柜王于贵愁眉苦脸地迎上来,“玉大人,你说我怎么就摊上这事。” “怎么了?”我有些...
    开封第一讲书人阅读 111,673评论 0 254
  • 文/不坏的土叔 我叫张陵,是天一观的道长。 经常有香客问我,道长,这世上最难降的妖魔是什么? 我笑而不...
    开封第一讲书人阅读 44,622评论 0 218
  • 正文 为了忘掉前任,我火速办了婚礼,结果婚礼上,老公的妹妹穿的比我还像新娘。我一直安慰自己,他们只是感情好,可当我...
    茶点故事阅读 53,047评论 3 295
  • 文/花漫 我一把揭开白布。 她就那样静静地躺着,像睡着了一般。 火红的嫁衣衬着肌肤如雪。 梳的纹丝不乱的头发上,一...
    开封第一讲书人阅读 40,974评论 1 224
  • 那天,我揣着相机与录音,去河边找鬼。 笑死,一个胖子当着我的面吹牛,可吹牛的内容都是我干的。 我是一名探鬼主播,决...
    沈念sama阅读 32,129评论 2 317
  • 文/苍兰香墨 我猛地睁开眼,长吁一口气:“原来是场噩梦啊……” “哼!你这毒妇竟也来了?” 一声冷哼从身侧响起,我...
    开封第一讲书人阅读 30,893评论 0 209
  • 序言:老挝万荣一对情侣失踪,失踪者是张志新(化名)和其女友刘颖,没想到半个月后,有当地人在树林里发现了一具尸体,经...
    沈念sama阅读 34,654评论 1 250
  • 正文 独居荒郊野岭守林人离奇死亡,尸身上长有42处带血的脓包…… 初始之章·张勋 以下内容为张勋视角 年9月15日...
    茶点故事阅读 30,828评论 2 254
  • 正文 我和宋清朗相恋三年,在试婚纱的时候发现自己被绿了。 大学时的朋友给我发了我未婚夫和他白月光在一起吃饭的照片。...
    茶点故事阅读 32,297评论 1 265
  • 序言:一个原本活蹦乱跳的男人离奇死亡,死状恐怖,灵堂内的尸体忽然破棺而出,到底是诈尸还是另有隐情,我是刑警宁泽,带...
    沈念sama阅读 28,619评论 3 262
  • 正文 年R本政府宣布,位于F岛的核电站,受9级特大地震影响,放射性物质发生泄漏。R本人自食恶果不足惜,却给世界环境...
    茶点故事阅读 33,326评论 3 243
  • 文/蒙蒙 一、第九天 我趴在偏房一处隐蔽的房顶上张望。 院中可真热闹,春花似锦、人声如沸。这庄子的主人今日做“春日...
    开封第一讲书人阅读 26,176评论 0 8
  • 文/苍兰香墨 我抬头看了看天上的太阳。三九已至,却和暖如春,着一层夹袄步出监牢的瞬间,已是汗流浃背。 一阵脚步声响...
    开封第一讲书人阅读 26,975评论 0 201
  • 我被黑心中介骗来泰国打工, 没想到刚下飞机就差点儿被人妖公主榨干…… 1. 我叫王不留,地道东北人。 一个月前我还...
    沈念sama阅读 36,118评论 2 285
  • 正文 我出身青楼,却偏偏与公主长得像,于是被迫代替她去往敌国和亲。 传闻我的和亲对象是个残疾皇子,可洞房花烛夜当晚...
    茶点故事阅读 35,909评论 2 278