feat: ignorePath option (#100)

Author: evilebottnawi

README.md

@@ -54,6 +54,7 @@ First thing first, let's make sure you have the necessary pre-requisites.
 | npmPkgJsonLint --configFile <file path> | -c | File path to local config file or module name. |
 | npmPkgJsonLint --quiet | -q | Report errors only |
 | npmPkgJsonLint --noConfigFiles | -ncf | Skips loading project config files (i.e. .npmpackagejsonlintrc.json and npmpackagejsonlint.config.js) |
+| npmPkgJsonLint --ignorePath | -i | Path to a file containing patterns that describe files to ignore. |
 
 ### Examples
 
@@ -99,6 +100,12 @@ $ npmPkgJsonLint --quiet ./packages
 
 > Looks for all `package.json` files in the `packages` directory. The CLI engine automatically looks for relevant config files for each package.json file that is found. Removes any warnings from the output using the long form for quieting output.
 
+```bash
+$ npmPkgJsonLint . --ignorePath .gitignore
+```
+
+> Looks for all `package.json` files in the project and exclude ignored paths. The CLI engine automatically looks for relevant config files for each package.json file that is found.
+
 ## Node.js API
 
 npm-package-json-lint exports two main objects: `CLIEngine` and `NpmPackageJsonLint`.
@@ -180,7 +187,8 @@ CLIEngine configuration object
 * `configFile`      {string}  Name of module/file to use.
 * `cwd`             {string}  The current working diretory for all file operations.
 * `useConfigFiles`  {boolean} False disables use of .npmpackagejsonlintrc.json files and npmpackagejsonlint.config.js files.
-* `rules`            {object} An object of rules to use.
+* `ignorePath`      {string} Path to a file containing patterns that describe files to ignore. The path can be absolute or relative to process.cwd(). By default, npm-package-json-lint looks for .npmpackagejsonlintignore in process.cwd().
+* `rules`           {object} An object of rules to use.
 
 ##### Example
 
@@ -193,6 +201,7 @@ const cliEngineOptions = {
   configFile: '',
   cwd: process.cwd(),
   useConfigFiles: true,
+  ignorePath: '',
   rules: {}
 };
 
@@ -220,6 +229,7 @@ const cliEngineOptions = {
   configFile: '',
   cwd: process.cwd(),
   useConfigFiles: true,
+  ignorePath: '',
   rules: {}
 };
 const patterns = ['.'];
@@ -267,6 +277,7 @@ const cliEngineOptions = {
   configFile: '',
   cwd: process.cwd(),
   useConfigFiles: true,
+  ignorePath: '',
   rules: {}
 };
 

package.json

@@ -37,6 +37,7 @@
     "ajv": "^6.7.0",
     "chalk": "^2.4.2",
     "glob": "^7.1.3",
+    "ignore": "^5.0.5",
     "is-path-inside": "^2.0.0",
     "is-plain-obj": "^1.1.0",
     "is-resolvable": "^1.1.0",

src/CLIEngine.js

@@ -7,10 +7,14 @@ const path = require('path');
 const NpmPackageJsonLint = require('./NpmPackageJsonLint');
 const Config = require('./Config');
 const glob = require('glob');
+const ignore = require('ignore');
 const ConfigValidator = require('./config/ConfigValidator');
 const Parser = require('./Parser');
 const pkg = require('../package.json');
 
+const DEFAULT_IGNORE_FILENAME = '.npmpackagejsonlintignore';
+const FILE_NOT_FOUND_ERROR_CODE = 'ENOENT';
+
 const noIssues = 0;
 
 /**
@@ -154,6 +158,31 @@ const isIssueAnError = function(issue) {
   return issue.severity === 'error';
 };
 
+/**
+ * Generates ignorer based on ignore file content.
+ *
+ * @param {String}            cwd     Current work directory.
+ * @param {CLIEngineOptions}  options CLIEngineOptions object.
+ * @returns {Object}          Ignorer
+ */
+const getIgnorer = function(cwd, options) {
+  const ignoreFilePath = options.ignorePath || DEFAULT_IGNORE_FILENAME;
+  const absoluteIgnoreFilePath = path.isAbsolute(ignoreFilePath)
+    ? ignoreFilePath
+    : path.resolve(cwd, ignoreFilePath);
+  let ignoreText = '';
+
+  try {
+    ignoreText = fs.readFileSync(absoluteIgnoreFilePath, 'utf8');
+  } catch (readError) {
+    if (readError.code !== FILE_NOT_FOUND_ERROR_CODE) {
+      throw readError;
+    }
+  }
+
+  return ignore().add(ignoreText);
+};
+
 /**
  * Generates a list of files to lint based on a list of provided patterns
  *
@@ -197,12 +226,13 @@ const getFileList = function(patterns, options) {
 
   const files = [];
   const addedFiles = new Set();
+  const ignorer = getIgnorer(cwd, options);
 
   globPatterns.forEach((pattern) => {
     const file = path.resolve(cwd, pattern);
 
     if (fs.existsSync(file) && fs.statSync(file).isFile()) {
-      if (addedFiles.has(file)) {
+      if (addedFiles.has(file) || ignorer.ignores(path.relative(cwd, file))) {
         return;
       }
 
@@ -225,7 +255,7 @@ const getFileList = function(patterns, options) {
       globFiles.forEach((globFile) => {
         const filePath = path.resolve(cwd, globFile);
 
-        if (addedFiles.has(filePath)) {
+        if (addedFiles.has(filePath) || ignorer.ignores(path.relative(cwd, filePath))) {
           return;
         }
 

src/cli.js

@@ -24,6 +24,7 @@ const cli = meow(`
         --quiet, -q Report errors only
         --noConfigFiles, -ncf Disables use of .npmpackagejsonlintrc.json files, npmpackagejsonlint.config.js files, and npmPackageJsonLintConfig object in package.json file.
         --configFile, -c File path of .npmpackagejsonlintrc.json
+        --ignorePath, -i Path to a file containing patterns that describe files to ignore. The path can be absolute or relative to process.cwd(). By default, npm-package-json-lint looks for .npmpackagejsonlintignore in process.cwd().
 
       Examples
         $ npmPkgJsonLint --version
@@ -34,6 +35,8 @@ const cli = meow(`
         $ npmPkgJsonLint --configFile ./config/npmpackagejsonlint.config.json .
         $ npmPkgJsonLint -q .
         $ npmPkgJsonLint --quiet ./packages
+        $ npmPkgJsonLint . --ignorePath .gitignore
+        $ npmPkgJsonLint . -i .gitignore
 `, {
   flags: {
     quiet: {
@@ -50,6 +53,11 @@ const cli = meow(`
       'type': 'string',
       'alias': 'c',
       'default': ''
+    },
+    ignorePath: {
+      'type': 'string',
+      'alias': 'i',
+      'default': ''
     }
   }
 });
@@ -73,6 +81,7 @@ const cliEngineOptions = {
   configFile: cli.flags.configFile,
   cwd: process.cwd(),
   useConfigFiles: !cli.flags.noConfigFiles,
+  ignorePath: cli.flags.ignorePath,
   rules: {}
 };
 

tests/fixtures/ignorePath/.gitignore-example

@@ -0,0 +1 @@
+ignoredDirectory

tests/fixtures/ignorePath/.npmpackagejsonlintrc.json

@@ -0,0 +1,42 @@
+{
+  "rules": {
+    "require-author": "error",
+    "require-description": "error",
+    "require-devDependencies": "error",
+    "require-homepage": "error",
+    "require-keywords": "error",
+    "require-name": "error",
+    "require-repository": "error",
+    "require-version": "error",
+    "description-type": "error",
+    "devDependencies-type": "error",
+    "homepage-type": "error",
+    "keywords-type": "error",
+    "name-type": "error",
+    "repository-type": "error",
+    "version-type": "error",
+    "valid-values-author": ["error", [
+      "Thomas Lindner"
+    ]],
+    "name-format": "error",
+    "version-format": "error",
+    "prefer-property-order": ["error", [
+      "name",
+      "version",
+      "description",
+      "keywords",
+      "homepage",
+      "bugs",
+      "author",
+      "repository",
+      "bin",
+      "files",
+      "main",
+      "scripts",
+      "dependencies",
+      "devDependencies",
+      "engines",
+      "license"
+    ]]
+  }
+}

tests/fixtures/ignorePath/ignoredDirectory/package.json

@@ -0,0 +1,16 @@
+{
+  "name": "npm-package-json-lint",
+  "version": "0.1.0",
+  "description": "CLI app for linting package.json files.",
+  "keywords": [
+    "lint"
+  ],
+  "homepage": "https://github.com/tclindner/npm-package-json-lint",
+  "repository": {
+    "type": "git",
+    "url": "https://github.com/tclindner/npm-package-json-lint"
+  },
+  "devDependencies": {
+    "mocha": "^2.4.5"
+  }
+}

tests/fixtures/ignorePath/package.json

@@ -0,0 +1,17 @@
+{
+  "name": "npm-package-json-lint",
+  "version": "0.1.0",
+  "description": "CLI app for linting package.json files.",
+  "keywords": [
+    "lint"
+  ],
+  "homepage": "https://github.com/tclindner/npm-package-json-lint",
+  "author": "Thomas Lindner",
+  "repository": {
+    "type": "git",
+    "url": "https://github.com/tclindner/npm-package-json-lint"
+  },
+  "devDependencies": {
+    "mocha": "^2.4.5"
+  }
+}

tests/fixtures/npmPackageJsonLintIgnore/.npmpackagejsonlintignore

@@ -0,0 +1 @@
+ignoredDirectory

tests/fixtures/npmPackageJsonLintIgnore/.npmpackagejsonlintrc.json

@@ -0,0 +1,42 @@
+{
+  "rules": {
+    "require-author": "error",
+    "require-description": "error",
+    "require-devDependencies": "error",
+    "require-homepage": "error",
+    "require-keywords": "error",
+    "require-name": "error",
+    "require-repository": "error",
+    "require-version": "error",
+    "description-type": "error",
+    "devDependencies-type": "error",
+    "homepage-type": "error",
+    "keywords-type": "error",
+    "name-type": "error",
+    "repository-type": "error",
+    "version-type": "error",
+    "valid-values-author": ["error", [
+      "Thomas Lindner"
+    ]],
+    "name-format": "error",
+    "version-format": "error",
+    "prefer-property-order": ["error", [
+      "name",
+      "version",
+      "description",
+      "keywords",
+      "homepage",
+      "bugs",
+      "author",
+      "repository",
+      "bin",
+      "files",
+      "main",
+      "scripts",
+      "dependencies",
+      "devDependencies",
+      "engines",
+      "license"
+    ]]
+  }
+}