Skip to content

Commit

Permalink
CLI shouldn't necc exit on semver check
Browse files Browse the repository at this point in the history
  • Loading branch information
paulirish authored Jul 8, 2016
1 parent e28ee77 commit dec44de
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions lighthouse-cli/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -23,10 +23,9 @@ const semver = require('semver');
const Printer = require('./printer');
const lighthouse = require('../lighthouse-core');

// node 5.x required due to use of ES2015 features
// node 5.x required due to use of ES2015 features, like spread operator
if (semver.lt(process.version, '5.0.0')) {
console.error('Lighthouse requires node version 5.0 or newer');
process.exit(1);
console.warn('Compatibility error', 'Lighthouse requires node 5+ or 4 with --harmony');
}

const cli = yargs
Expand Down

0 comments on commit dec44de

Please sign in to comment.