checkPlugin: Move log messages where they make more sense

pull/4641/head
Richard Hansen 2021-01-09 00:59:20 -05:00 committed by John McLear
parent d816bb2b38
commit 102c01f723
1 changed files with 2 additions and 2 deletions

View File

@ -433,9 +433,9 @@ fs.readdir(pluginPath, (err, rootFiles) => {
// linting begins // linting begins
try { try {
console.log('Linting...');
const lintCmd = autoFix ? 'npx eslint --fix .' : 'npx eslint'; const lintCmd = autoFix ? 'npx eslint --fix .' : 'npx eslint';
execSync(lintCmd); execSync(lintCmd);
console.log('Linting...');
if (autoFix) { if (autoFix) {
// todo: if npm run lint doesn't do anything no need for... // todo: if npm run lint doesn't do anything no need for...
hasAutoFixed = true; hasAutoFixed = true;
@ -447,7 +447,6 @@ fs.readdir(pluginPath, (err, rootFiles) => {
// linting ends. // linting ends.
if (hasAutoFixed) { if (hasAutoFixed) {
console.log('Fixes applied, please check git diff then run the following command:\n\n');
// bump npm Version // bump npm Version
const cmd = [ const cmd = [
'git rm -rf node_modules --ignore-unmatch', 'git rm -rf node_modules --ignore-unmatch',
@ -460,6 +459,7 @@ fs.readdir(pluginPath, (err, rootFiles) => {
console.log('Attempting autocommit and auto publish to npm'); console.log('Attempting autocommit and auto publish to npm');
execSync(cmd); execSync(cmd);
} else { } else {
console.log('Fixes applied, please check git diff then run the following command:');
console.log(`(cd node_modules/${pluginName} && ${cmd})`); console.log(`(cd node_modules/${pluginName} && ${cmd})`);
} }
} }