fix: actually lint staged files on commit (#44036)

This commit is contained in:
Oliver Eyton-Williams
2021-10-28 12:02:10 +02:00
committed by GitHub
parent 07bfe87419
commit 2d5305af7f

View File

@ -5,14 +5,18 @@ const cli = new ESLint();
// if a lot of files are changed, it's faster to run prettier/eslint on the // if a lot of files are changed, it's faster to run prettier/eslint on the
// whole project than to run them on each file separately // whole project than to run them on each file separately
module.exports = { module.exports = {
'*.(js|ts|tsx)': files => '*.(js|ts|tsx)': async files => {
files.length > 10 const ignoredIds = await Promise.all(
files.map(file => cli.isPathIgnored(file))
);
const lintableFiles = files.filter((_, i) => !ignoredIds[i]);
return files.length > 10
? ['eslint --max-warnings=0 --cache --fix .', 'prettier --write .'] ? ['eslint --max-warnings=0 --cache --fix .', 'prettier --write .']
: [ : [
'eslint --max-warnings=0 --cache --fix ' + 'eslint --max-warnings=0 --cache --fix ' + lintableFiles.join(' '),
files.filter(file => !cli.isPathIgnored(file)).join(' '),
...files.map(filename => `prettier --write '${filename}'`) ...files.map(filename => `prettier --write '${filename}'`)
], ];
},
'*.!(js|ts|tsx)': files => '*.!(js|ts|tsx)': files =>
files.length > 10 files.length > 10
? 'prettier --write .' ? 'prettier --write .'