zoukankan      html  css  js  c++  java
  • pre-commit钩子,代码质量检查

    目前基本使用三款js代码质量检查工具: jslint, jshint, eslint。许多IDE里面也有对应的检查插件,在每次ctrl + s 保存文件的时候,检查当前文件是否符合规范,保证代码质量。
    许多团队都会指定一套代码规范code review,更加严格的检查每次代码修改。 也可以在git commit之前,检查代码,保证所有提交到版本库中的代码都是符合规范的,

    在看vue源码时,不免修改代码,就会触发里面配置好的钩子函数。于是,仔细研究了一下vue配置方法,可以发现配置非常简单。

    git 钩子文档上介绍非常详细,git init后,在.git/hooks文件中,有一些.simple结尾的钩子示例脚本,如果想启用对应的钩子函数,只需手动删除后缀。所以,列出两种配置方法:

    1. 手动修改钩子文件

    按照文档上,配置钩子脚本,修改hooks中文件名对应的钩子文件,启用钩子。使用shell脚本检查,可以参考vue1.x 里面如何使用

    ``` !/usr/bin/env bash
     # get files to be linted
    FILES=$(git diff --cached --name-only | grep -E '^src|^test/unit/specs|^test/e2e')
    
     # lint them if any
    if [[ $FILES ]]; then
      ./node_modules/.bin/eslint $FILES
    fi
    
    
    <p>文件名是<code>pre-commit</code>,在commit 之前启用的钩子函数, 利用 <code>git diff</code>查看当前有哪些文件修改过,只对指定文件夹中修改的文件使用eslint进行代码检查,渐进式对整个项目实现代码规范。</p>
    <p>脚本写好后,不用每次都手动复制到<code>.git/hooks</code>目录下,只需对当前文件创建软连接,到指定目录,<a href="https://github.com/vuejs/vue/blob/v1.0.26/package.json#L29" rel="nofollow noreferrer">在package.json中配置脚本命令</a>,</p>
    ```"scripts": {
       "install-hook": "ln -s ../../build/git-hooks/pre-commit .git/hooks/pre-commit",
    }
    

    在项目初始化后, 执行npm run install-hook,很方便地配置好了pre-commit 钩子

    2. 利用yorkie or husky + lint-staged 构建钩子

    在 vue最新的版本中,已经使用尤大改写的youkie, youkie实际是fork husky,然后做了一些定制化的改动, 使得钩子能从package.json的 "gitHooks"属性中读取,

    ```{ "gitHooks": { "pre-commit": "foo" } } ```

    使用方法跟husky 类似,可以查看husky 文档,介绍非常详细。

    
     npm install husky --save-dev
     # or npm install yorkie --save-dev
    

    安装完成后,可以发现已经改写了hooks 目录中的文件,只需在package.json 中配置对应钩子要执行的脚本。
    husky 配置:

    
    // package.json
    {
      "husky": {
        "hooks": {
          "pre-commit": "npm test",
          "pre-push": "npm test",
          "...": "..."
        }
      }
    }
    

    回头看看,vue中如何配置

    
    // package.json
     "gitHooks": {
        "pre-commit": "lint-staged",
        "commit-msg": "node scripts/verify-commit-msg.js"
      }
     "lint-staged": {
        "*.js": [
          "eslint --fix",
          "git add"
        ]
      }
    
    

    前面提到,利用git diff,只lint当前改动的文件,lint-staged就非常准确的解决了这一问题,从这个包名,就可以看出,Run linters on git staged files,只针对改动的文件进行处理。
    结合husky一起使用,安装依赖:

    ```npm install --save-dev lint-staged husky ```

    修改package.json 文件

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

    使用了eslint,需要配置.eslintrc, lint-staged还有一个好处,可以在lint后,更加灵活,执行其他脚本,尝试进行修改错误,比如 eslint --fix 检查后并修复错误。

    上面列出的vue 文件使用了类似的配置,另外增加了 commit-msg 钩子,对提交说明进行检查,在 scripts/verify-commit-msg.js文件中可以找到检查脚本,

    
    const chalk = require('chalk')
    const msgPath = process.env.GIT_PARAMS
    const msg = require('fs').readFileSync(msgPath, 'utf-8').trim()
    
    const commitRE = /^(revert: )?(feat|fix|polish|docs|style|refactor|perf|test|workflow|ci|chore|types|build)((.+))?: .{1,50}/
    
    if (!commitRE.test(msg)) {
      console.log()
      console.error(
        `  ${chalk.bgRed.white(' ERROR ')} ${chalk.red(`invalid commit message format.`)}
    
    ` +
        chalk.red(`  Proper commit message format is required for automated changelog generation. Examples:
    
    `) +
        `    ${chalk.green(`feat(compiler): add 'comments' option`)}
    ` +
        `    ${chalk.green(`fix(v-model): handle events on blur (close #28)`)}
    
    ` +
        chalk.red(`  See .github/COMMIT_CONVENTION.md for more details.
    `) +
        chalk.red(`  You can also use ${chalk.cyan(`npm run commit`)} to interactively generate a commit message.
    `)
      )
      process.exit(1)
    }
    

    利用process.env.GIT_PARAMS 找到目录,读取msg 说明,进行检查。

    使用 husky 要注意,对应属性名已经改为HUSKY_GIT_PARAMS , 而不是原始的 GIT_PARAMS 环境变量。

    来源:https://segmentfault.com/a/1190000016750078

  • 相关阅读:
    JS使用Enter事件将输入的字符倒叙输出
    博客园 页面定制CSS代码
    C#将DataTable数据导出CSV文件
    js时间格式转换
    将十进制数据转换为二进制AND将二进制数据转换为十进制!
    ref和out的使用与区别
    形参和实参的区别
    为啥Underlay才是容器网络的最佳落地选择
    你问我答:容器篇(1)
    切忌一步到位,谈谈DevOps实施落地
  • 原文地址:https://www.cnblogs.com/datiangou/p/10130557.html
Copyright © 2011-2022 走看看