checkPlugin: Make `autocommit` imply `autoupdate`, `autoupdate` imply `autofix`

image-example
Richard Hansen 2021-01-10 18:35:50 -05:00 committed by John McLear
parent 27df9df47f
commit 039c50de80
2 changed files with 11 additions and 19 deletions

View File

@ -24,7 +24,7 @@ node bin/plugins/checkPlugin.js ep_whatever autofix
## Autocommitting, push, npm minor patch and npm publish (highly dangerous) ## Autocommitting, push, npm minor patch and npm publish (highly dangerous)
``` ```
node bin/plugins/checkPlugin.js ep_whatever autofix autocommit node bin/plugins/checkPlugin.js ep_whatever autocommit
``` ```
# All the plugins # All the plugins
@ -41,7 +41,7 @@ cd ..
for dir in node_modules/ep_*; do for dir in node_modules/ep_*; do
dir=${dir#node_modules/} dir=${dir#node_modules/}
[ "$dir" != ep_etherpad-lite ] || continue [ "$dir" != ep_etherpad-lite ] || continue
node bin/plugins/checkPlugin.js "$dir" autofix autocommit node bin/plugins/checkPlugin.js "$dir" autocommit
done done
``` ```

View File

@ -1,15 +1,13 @@
'use strict'; 'use strict';
/* /*
* * Usage -- see README.md
* Usage -- see README.md *
* * Normal usage: node bin/plugins/checkPlugin.js ep_whatever
* Normal usage: node bin/plugins/checkPlugin.js ep_whatever * Auto fix the things it can: node bin/plugins/checkPlugin.js ep_whatever autofix
* Auto fix the things it can: node bin/plugins/checkPlugin.js ep_whatever autofix * Auto commit, push and publish to npm (highly dangerous):
* Auto commit, push and publish(to npm) * highly dangerous: * node bin/plugins/checkPlugin.js ep_whatever autocommit
node bin/plugins/checkPlugin.js ep_whatever autofix autocommit */
*/
const fs = require('fs'); const fs = require('fs');
const childProcess = require('child_process'); const childProcess = require('child_process');
@ -27,15 +25,9 @@ const pluginPath = `node_modules/${pluginName}`;
console.log(`Checking the plugin: ${pluginName}`); console.log(`Checking the plugin: ${pluginName}`);
const optArgs = process.argv.slice(3); const optArgs = process.argv.slice(3);
// Should we autofix?
const autoFix = optArgs.indexOf('autofix') !== -1;
// Should we update files where possible?
const autoUpdate = optArgs.indexOf('autoupdate') !== -1;
// Should we automcommit and npm publish?!
const autoCommit = optArgs.indexOf('autocommit') !== -1; const autoCommit = optArgs.indexOf('autocommit') !== -1;
const autoUpdate = autoCommit || optArgs.indexOf('autoupdate') !== -1;
const autoFix = autoUpdate || optArgs.indexOf('autofix') !== -1;
const execSync = (cmd, opts = {}) => (childProcess.execSync(cmd, { const execSync = (cmd, opts = {}) => (childProcess.execSync(cmd, {
cwd: `${pluginPath}/`, cwd: `${pluginPath}/`,