Require Node.js 8

.travis.yml

@@ -2,4 +2,3 @@ language: node_js
 node_js:
   - '10'
   - '8'
-  - '6'

index.js

@@ -13,10 +13,11 @@ class CancelError extends Error {
 
 class PCancelable {
 	static fn(userFn) {
-		return (...args) => {
+		return (...arguments_) => {
 			return new PCancelable((resolve, reject, onCancel) => {
-				args.push(onCancel);
-				userFn(...args).then(resolve, reject);
+				arguments_.push(onCancel);
+				// eslint-disable-next-line promise/prefer-await-to-then
+				userFn(...arguments_).then(resolve, reject);
 			});
 		};
 	}
@@ -51,8 +52,8 @@ class PCancelable {
 			Object.defineProperties(onCancel, {
 				shouldReject: {
 					get: () => this._rejectOnCancel,
-					set: bool => {
-						this._rejectOnCancel = bool;
+					set: boolean => {
+						this._rejectOnCancel = boolean;
 					}
 				}
 			});
@@ -62,6 +63,7 @@ class PCancelable {
 	}
 
 	then(onFulfilled, onRejected) {
+		// eslint-disable-next-line promise/prefer-await-to-then
 		return this._promise.then(onFulfilled, onRejected);
 	}
 

package.json

@@ -10,7 +10,7 @@
 		"url": "sindresorhus.com"
 	},
 	"engines": {
-		"node": ">=6"
+		"node": ">=8"
 	},
 	"scripts": {
 		"test": "xo && ava && tsd"

test.js

@@ -3,6 +3,7 @@ import delay from 'delay';
 import promiseFinally from 'promise.prototype.finally';
 import PCancelable from '.';
 
+// TODO: Remove this when targeting Node.js 10
 promiseFinally.shim();
 
 const fixture = Symbol('fixture');