diff --git a/scripts/tasks/danger.js b/scripts/tasks/danger.js index e85dba26219..157a75ac49c 100644 --- a/scripts/tasks/danger.js +++ b/scripts/tasks/danger.js @@ -31,9 +31,7 @@ spawn( ).on('close', function(code) { if (code !== 0) { console.error('Danger failed'); - } else { - console.log('Danger passed'); - } + } else {} process.exit(code); }); diff --git a/scripts/tasks/eslint.js b/scripts/tasks/eslint.js index 0205873f0aa..5e781cd4d84 100644 --- a/scripts/tasks/eslint.js +++ b/scripts/tasks/eslint.js @@ -11,19 +11,13 @@ const minimist = require('minimist'); const runESLint = require('../eslint'); async function main() { - console.log('Linting all files...'); // https://circleci.com/docs/2.0/env-vars/#circleci-environment-variable-descriptions - if (!process.env.CI) { - console.log('Hint: run `yarn linc` to only lint changed files.'); - } + if (!process.env.CI) {} // eslint-disable-next-line no-unused-vars const {_, ...cliOptions} = minimist(process.argv.slice(2)); - if (await runESLint({onlyChanged: false, ...cliOptions})) { - console.log('Lint passed.'); - } else { - console.log('Lint failed.'); + if (await runESLint({onlyChanged: false, ...cliOptions})) {} else { process.exit(1); } } diff --git a/scripts/tasks/flow-ci.js b/scripts/tasks/flow-ci.js index 99099270cf4..48785b17951 100644 --- a/scripts/tasks/flow-ci.js +++ b/scripts/tasks/flow-ci.js @@ -28,7 +28,6 @@ async function checkAll() { const rendererInfo = inlinedHostConfigs[i]; if (rendererInfo.isFlowTyped) { await runFlow(rendererInfo.shortName, ['check']); - console.log(); } } } diff --git a/scripts/tasks/flow.js b/scripts/tasks/flow.js index a9543743732..ff13dcd2eba 100644 --- a/scripts/tasks/flow.js +++ b/scripts/tasks/flow.js @@ -22,30 +22,9 @@ const primaryRenderer = inlinedHostConfigs.find( info => info.isFlowTyped && info.shortName === process.argv[2] ); if (!primaryRenderer) { - console.log( - 'The ' + - chalk.red('yarn flow') + - ' command now requires you to pick a primary renderer:' - ); - console.log(); inlinedHostConfigs.forEach(rendererInfo => { - if (rendererInfo.isFlowTyped) { - console.log(' * ' + chalk.cyan('yarn flow ' + rendererInfo.shortName)); - } + if (rendererInfo.isFlowTyped) {} }); - console.log(); - console.log('If you are not sure, run ' + chalk.green('yarn flow dom') + '.'); - console.log( - 'This will still typecheck non-DOM packages, although less precisely.' - ); - console.log(); - console.log('Note that checks for all renderers will run on CI.'); - console.log( - 'You can also do this locally with ' + - chalk.cyan('yarn flow-ci') + - ' but it will be slow.' - ); - console.log(); process.exit(1); } diff --git a/scripts/tasks/linc.js b/scripts/tasks/linc.js index e3b89237528..9335c1da22f 100644 --- a/scripts/tasks/linc.js +++ b/scripts/tasks/linc.js @@ -11,15 +11,10 @@ const minimist = require('minimist'); const runESLint = require('../eslint'); async function main() { - console.log('Linting changed files...'); - // eslint-disable-next-line no-unused-vars const {_, ...cliOptions} = minimist(process.argv.slice(2)); - if (await runESLint({onlyChanged: true, ...cliOptions})) { - console.log('Lint passed for changed files.'); - } else { - console.log('Lint failed for changed files.'); + if (await runESLint({onlyChanged: true, ...cliOptions})) {} else { process.exit(1); } } diff --git a/scripts/tasks/version-check.js b/scripts/tasks/version-check.js index 57dcfdfba63..d7bee0a73f2 100644 --- a/scripts/tasks/version-check.js +++ b/scripts/tasks/version-check.js @@ -28,12 +28,6 @@ Object.keys(versions).forEach(function(name) { const version = versions[name]; if (version !== reactVersion) { allVersionsMatch = false; - console.log( - '%s version does not match package.json. Expected %s, saw %s.', - name, - reactVersion, - version - ); } });