Skip to content
Permalink
Browse files

doc, tools: add doc linting to CI

PR-URL: #12640
Fixes: #12635
Refs: #12563
Reviewed-By: Teddy Katz <teddy.katz@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
  • Loading branch information...
vsemozhetbyt committed Apr 25, 2017
1 parent 71abfa9 commit 42dca99cd73eb26c24327e3c7e8162d150a91162
Showing with 5 additions and 3 deletions.
  1. +1 −1 Makefile
  2. +3 −1 tools/jslint.js
  3. +1 −1 vcbuild.bat
@@ -861,7 +861,7 @@ jslint:
jslint-ci:
@echo "Running JS linter..."
$(NODE) tools/jslint.js $(PARALLEL_ARGS) -f tap -o test-eslint.tap \
benchmark lib test tools
benchmark doc lib test tools

CPPLINT_EXCLUDE ?=
CPPLINT_EXCLUDE += src/node_root_certs.h
@@ -1,6 +1,7 @@
'use strict';

const rulesDirs = ['tools/eslint-rules'];
const extensions = ['.js', '.md'];
// This is the maximum number of files to be linted per worker at any given time
const maxWorkload = 40;

@@ -14,7 +15,8 @@ const glob = require('./eslint/node_modules/glob');

const cwd = process.cwd();
const cliOptions = {
rulePaths: rulesDirs
rulePaths: rulesDirs,
extensions: extensions,
};

// Check if we should fix errors that are fixable
@@ -428,7 +428,7 @@ goto exit

:jslint-ci
echo running jslint-ci
%config%\node tools\jslint.js -J -f tap -o test-eslint.tap benchmark lib test tools
%config%\node tools\jslint.js -J -f tap -o test-eslint.tap benchmark doc lib test tools
goto exit

:no-lint

0 comments on commit 42dca99

Please sign in to comment.
You can’t perform that action at this time.