Ignore paths from .gitignore

options-manager.js

@@ -7,6 +7,8 @@ const deepAssign = require('deep-assign');
 const multimatch = require('multimatch');
 const resolveFrom = require('resolve-from');
 const pathExists = require('path-exists');
+const parseGitignore = require('parse-gitignore');
+const globby = require('globby');
 
 const DEFAULT_IGNORE = [
 	'**/node_modules/**',
@@ -191,10 +193,33 @@ function groupConfigs(paths, baseOptions, overrides) {
 	return arr;
 }
 
+function getIgnores(opts) {
+	const gitignores = globby.sync('**/.gitignore');
+	let ignores = [];
+
+	gitignores.forEach(path => {
+		const result = parseGitignore(path);
+		const location = path.substring(0, path.length - 10);
+		const fullPathResult = [];
+
+		result.forEach(file => {
+			file = location + file;
+			fullPathResult.push(file);
+		});
+
+		ignores = ignores.concat(fullPathResult || []);
+	});
+
+	opts.ignores = DEFAULT_IGNORE.concat(opts.ignores || [], ignores);
+
+	return opts;
+}
+
 function preprocess(opts) {
 	opts = mergeWithPkgConf(opts);
 	opts = normalizeOpts(opts);
-	opts.ignores = DEFAULT_IGNORE.concat(opts.ignores || []);
+	opts = getIgnores(opts);
+
 	return opts;
 }
 
@@ -208,3 +233,4 @@ exports.mergeApplicableOverrides = mergeApplicableOverrides;
 exports.groupConfigs = groupConfigs;
 exports.preprocess = preprocess;
 exports.emptyOptions = emptyOptions;
+exports.getIgnores = getIgnores;

package.json

@@ -77,6 +77,7 @@
     "has-flag": "^2.0.0",
     "meow": "^3.4.2",
     "multimatch": "^2.1.0",
+    "parse-gitignore": "^0.3.1",
     "path-exists": "^3.0.0",
     "pkg-conf": "^1.0.1",
     "resolve-cwd": "^1.0.0",

readme.md

@@ -159,7 +159,7 @@ Additional global variables your code accesses during execution.
 
 Type: `Array`
 
-Some [paths](https://github.com/sindresorhus/xo/blob/master/options-manager.js) are ignored by default. Additional ignores can be added here.
+Some [paths](https://github.com/sindresorhus/xo/blob/master/options-manager.js) are ignored by default, including paths in .gitignore. Additional ignores can be added here.
 
 ### space
 

test/.gitignore

@@ -0,0 +1,2 @@
+# This is a test .gitignore.
+foo/

test/bar/.gitignore

@@ -0,0 +1 @@
+foo.js

test/cli.js

@@ -31,6 +31,17 @@ test.failing('ignores fixture', async t => {
 	t.throws(execa('../../../cli.js', ['--no-local'], {cwd}));
 });
 
+test('ignore files in .gitignore', async t => {
+	const cwd = path.join(__dirname, 'fixtures/gitignore');
+
+	try {
+		await execa('../../../cli.js', ['--no-local'], {cwd});
+	} catch (err) {
+		t.is(err.stdout.indexOf('foo.js'), -1);
+		t.true(err.stdout.indexOf('bar.js') !== -1);
+	}
+});
+
 test('supports being extended with a shareable config', async () => {
 	const cwd = path.join(__dirname, 'fixtures/project');
 	await execa('../../../cli.js', ['--no-local'], {cwd});

test/fixtures/gitignore/index.js

@@ -0,0 +1,5 @@
+'use strict'
+
+module.exports = function (foo) {
+  return foo + 'bar'
+}

test/fixtures/gitignore/test/.gitignore

@@ -0,0 +1 @@
+foo.js

test/fixtures/gitignore/test/bar.js

@@ -0,0 +1,6 @@
+import test from 'ava'
+import fn from '../'
+
+test(t => {
+   t.is(fn('foo'), fn('foobar'))
+})

test/fixtures/gitignore/test/foo.js

@@ -0,0 +1,6 @@
+import test from 'ava'
+import fn from '../'
+
+test(t => {
+    t.is(fn('foo'), fn('foobar'))
+})

test/options-manager.js

@@ -136,3 +136,9 @@ test('groupConfigs', t => {
 		return obj;
 	}));
 });
+
+test('gitignore', t => {
+	const result = manager.getIgnores({});
+	t.true(result.ignores.includes('foo/**'));
+	t.true(result.ignores.includes('bar/foo.js'));
+});