Skip to content

Commit

Permalink
Increase minimum Node version to 6
Browse files Browse the repository at this point in the history
Node 4 loses LTS status on 2018-04-30.
  • Loading branch information
malept committed Apr 5, 2018
1 parent 6712b9b commit 5cd3de6
Show file tree
Hide file tree
Showing 5 changed files with 9 additions and 5 deletions.
4 changes: 4 additions & 0 deletions NEWS.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,10 @@

[Unreleased]: https://github.com/electron-userland/electron-installer-snap/compare/v2.0.1...master

## Changed

* Minimum Node version increased to Node 6, as Node 4 is no longer LTS

## [2.0.1] - 2018-02-22

[2.0.1]: https://github.com/electron-userland/electron-installer-snap/compare/v2.0.0...v2.0.1
Expand Down
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ Builds Snap files for Electron applications that have already been bundled and c

## Requirements

Requires Node 4 or greater, and [`snapcraft`](https://snapcraft.io).
Requires Node 6 or greater, and [`snapcraft`](https://snapcraft.io).

## Quick Start

Expand Down
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,7 @@
"yargs": "^11.0.0"
},
"engines": {
"node": ">= 4.0"
"node": ">= 6.0"
},
"eslintConfig": {
"extends": [
Expand Down
2 changes: 1 addition & 1 deletion src/cli.js
Original file line number Diff line number Diff line change
Expand Up @@ -80,7 +80,7 @@ function parseArgs () {
const YARGS_KEYS = ['_', '$0', 'help']
for (const key in args) {
const value = args[key]
if (value !== undefined && key.indexOf('-') === -1 && YARGS_KEYS.indexOf(key) === -1) {
if (value !== undefined && !key.includes('-') && !YARGS_KEYS.includes(key)) {
filteredArgs[key] = value
}
}
Expand Down
4 changes: 2 additions & 2 deletions test/_util.js
Original file line number Diff line number Diff line change
Expand Up @@ -20,11 +20,11 @@ const test = require('ava')
const tmp = require('tmp-promise')

function assertIncludes (t, collection, value, message) {
return t.not(collection.indexOf(value), -1, message)
return t.true(collection.includes(value), message)
}

function assertNotIncludes (t, collection, value, message) {
return t.is(collection.indexOf(value), -1, message)
return t.false(collection.includes(value), message)
}

test.beforeEach(t => {
Expand Down

0 comments on commit 5cd3de6

Please sign in to comment.