17

我想在提交到 Mercurial 或 Git 存储库之前运行 JSLint。

我希望这是一个自动设置的步骤，而不是依赖开发人员（主要是我）记住事先运行 JSLint。我通常在开发时运行 JSLint，但想在提交到 repo 之前通过 JSLint 的 JS 文件指定一个合同。

对于 Mercurial，此页面说明了预提交语法，但似乎唯一可用的变量是提交中涉及的 parent1 和 parent2 变更集 ID。我真正想要的是与提交有关的文件名列表，这样我就可以选择 .js 文件并在它们上运行 jslint。

GIT 的类似问题，作为预提交脚本的一部分可用的默认信息似乎有限。

可能可行的是调用 hg status/git status 作为预提交脚本的一部分，解析该输出以查找 JS 文件，然后以这种方式完成工作。不过，我希望有一些更简单的东西，我不确定作为预提交钩子的一部分调用状态是否反映了正确的信息。例如，在 Git 中，如果尚未添加更改文件，但 git commit 使用 -a，这些文件是否会作为提交集的一部分显示在 git status 输出的正确部分？

更新：我得到了一些工作，在这里可以看到：http: //github.com/jrburke/dvcs_jslint/

4

3 回答 3

12

以下是@Bitbieger 的 Git 解决方案的变体，它与Node.js和node-jslint的本地副本一起工作（即您需要npm install jslint在根存储库目录中）。

另外脚本：

  • 对所有 .html 和 .json 文件以及 .js 运行 jslint
  • 仅对已添加、复制或修改的文件运行 jslint。这可以防止 jslint 在已重命名或删除的文件上出错。
  • 复制任何 jslint 错误供用户查看
  • 使用--indent 4 --white truejslint 选项来确保源代码的一致性

要使其正常工作，请将以下内容复制到.git/hooks/pre-commit并且不要忘记chmod +x .git/hooks/pre-commit

# Pre-commit hook passing files through jslint
#
# This ensures that all js, html and json files are valid and conform
# to expectations.

ROOT_DIR=$(git rev-parse --show-toplevel)
JSLINT="${ROOT_DIR}/node_modules/.bin/jslint --indent 4 --white true"

for file in $(git diff-index --name-only --diff-filter=ACM --cached HEAD -- | grep -P '\.((js)|(html)|(json))$'); do
    if node $JSLINT $file 2>&1 | grep 'No errors found' ; then
        echo "jslint passed ${file}"
        exit 0
    else
        node $JSLINT $file
        exit 1
    fi  
done
于 2011-09-12T09:45:22.790 回答
2

对于 git，.git/hooks 目录中有示例。如果您只需要 JSLint 的文件名，您可以使用git diff --name-only，在我的示例中，它将列出与当前HEAD.

于 2009-12-03T10:26:24.767 回答
1

JSLint 与 SpiderMonkey

for js in $(git diff-index --name-only --cached HEAD -- | grep '\.js$'); do
    if jslint.sh $js 2>&1 | grep 'Lint at line' ; then
        echo $js
        exit 1
    else
        echo "js files validated"
        exit 0
    fi  
done
于 2009-12-06T11:15:20.687 回答