refactor: switch to es6 code standards

Author: keithamus

bench/index.js

@@ -1,9 +1,7 @@
-'use strict';
-
-var typeDetect = require('../');
-var Benchmark = require('benchmark');
-var benches = [];
-var fixtures = {
+import typeDetect from '../';
+import Benchmark from 'benchmark';
+const benches = [];
+const fixtures = {
   'string literal    ': '',
   'array literal     ': [],
   'boolean literal   ': true,
@@ -14,7 +12,7 @@ var fixtures = {
   'promise           ': Promise.resolve(),
   'null              ': null,
   'undefined         ': undefined,
-  'function          ': function () {},
+  'function          '() {},
 
   'buffer            ': new Buffer(1),
   'date              ': new Date(),
@@ -44,7 +42,7 @@ try {
   'Uint32Array', 'Uint16Array', 'Uint8Array',
   'Int32Array', 'Int16Array', 'Int8Array',
   'Uint8ClampedArray',
-].forEach(function (value) {
+].forEach((value) => {
   if (typeof global[value] === 'function') {
     fixtures[value + new Array(19 - value.length).join(' ')] = new (global[value])(1);
   }
@@ -53,15 +51,13 @@ if (typeof DataView === 'function') {
   fixtures['DataView          '] = new DataView(new ArrayBuffer(1));
 }
 
-var filter = process.argv[2] || '';
-Object.keys(fixtures).filter(function (key) {
-  return key.indexOf(filter) !== -1;
-}).forEach(function (test) {
+const filter = process.argv[2] || '';
+Object.keys(fixtures).filter((key) => key.indexOf(filter) !== -1).forEach((test) => {
   benches.push(new Benchmark(test, {
-    fn: function () {
+    fn() {
       typeDetect(fixtures[test]);
     },
-    onCycle: function (event) {
+    onCycle(event) {
       process.stdout.clearLine();
       process.stdout.cursorTo(0);
       process.stdout.write(event.target.toString());

index.js

@@ -1,43 +1,41 @@
-'use strict';
-
 /* !
  * type-detect
  * Copyright(c) 2013 jake luer <[email protected]>
  * MIT Licensed
  */
-var promiseExists = typeof Promise === 'function';
+const promiseExists = typeof Promise === 'function';
 
 /* eslint-disable no-undef */
-var globalObject = typeof self === 'object' ? self : global; // eslint-disable-line id-blacklist
+const globalObject = typeof self === 'object' ? self : global; // eslint-disable-line id-blacklist
 
 /*
  * All of these attributes must be available on the global object for the current environment
  * to be considered a DOM environment (browser)
  */
-var isDom = typeof window === 'object' &&
+const isDom = typeof window === 'object' &&
   'document' in window &&
   'navigator' in window &&
   'HTMLElement' in window;
 /* eslint-enable */
 
-var symbolExists = typeof Symbol !== 'undefined';
-var mapExists = typeof Map !== 'undefined';
-var setExists = typeof Set !== 'undefined';
-var weakMapExists = typeof WeakMap !== 'undefined';
-var weakSetExists = typeof WeakSet !== 'undefined';
-var dataViewExists = typeof DataView !== 'undefined';
-var symbolIteratorExists = symbolExists && typeof Symbol.iterator !== 'undefined';
-var symbolToStringTagExists = symbolExists && typeof Symbol.toStringTag !== 'undefined';
-var setEntriesExists = setExists && typeof Set.prototype.entries === 'function';
-var mapEntriesExists = mapExists && typeof Map.prototype.entries === 'function';
-var setIteratorPrototype = setEntriesExists && Object.getPrototypeOf(new Set().entries());
-var mapIteratorPrototype = mapEntriesExists && Object.getPrototypeOf(new Map().entries());
-var arrayIteratorExists = symbolIteratorExists && typeof Array.prototype[Symbol.iterator] === 'function';
-var arrayIteratorPrototype = arrayIteratorExists && Object.getPrototypeOf([][Symbol.iterator]());
-var stringIteratorExists = symbolIteratorExists && typeof String.prototype[Symbol.iterator] === 'function';
-var stringIteratorPrototype = stringIteratorExists && Object.getPrototypeOf(''[Symbol.iterator]());
-var toStringLeftSliceLength = 8;
-var toStringRightSliceLength = -1;
+const symbolExists = typeof Symbol !== 'undefined';
+const mapExists = typeof Map !== 'undefined';
+const setExists = typeof Set !== 'undefined';
+const weakMapExists = typeof WeakMap !== 'undefined';
+const weakSetExists = typeof WeakSet !== 'undefined';
+const dataViewExists = typeof DataView !== 'undefined';
+const symbolIteratorExists = symbolExists && typeof Symbol.iterator !== 'undefined';
+const symbolToStringTagExists = symbolExists && typeof Symbol.toStringTag !== 'undefined';
+const setEntriesExists = setExists && typeof Set.prototype.entries === 'function';
+const mapEntriesExists = mapExists && typeof Map.prototype.entries === 'function';
+const setIteratorPrototype = setEntriesExists && Object.getPrototypeOf(new Set().entries());
+const mapIteratorPrototype = mapEntriesExists && Object.getPrototypeOf(new Map().entries());
+const arrayIteratorExists = symbolIteratorExists && typeof Array.prototype[Symbol.iterator] === 'function';
+const arrayIteratorPrototype = arrayIteratorExists && Object.getPrototypeOf([][Symbol.iterator]());
+const stringIteratorExists = symbolIteratorExists && typeof String.prototype[Symbol.iterator] === 'function';
+const stringIteratorPrototype = stringIteratorExists && Object.getPrototypeOf(''[Symbol.iterator]());
+const toStringLeftSliceLength = 8;
+const toStringRightSliceLength = -1;
 /**
  * ### typeOf (obj)
  *
@@ -48,7 +46,7 @@ var toStringRightSliceLength = -1;
  * @return {String} object type
  * @api public
  */
-module.exports = function typeDetect(obj) {
+export default function typeDetect(obj) {
   /* ! Speed optimisation
    * Pre:
    *   string literal     x 3,039,035 ops/sec ±1.62% (78 runs sampled)
@@ -63,7 +61,7 @@ module.exports = function typeDetect(obj) {
    *   undefined          x 32,363,368 ops/sec ±1.07% (82 runs sampled)
    *   function           x 31,296,870 ops/sec ±0.96% (83 runs sampled)
    */
-  var typeofObj = typeof obj;
+  const typeofObj = typeof obj;
   if (typeofObj !== 'object') {
     return typeofObj;
   }
@@ -231,12 +229,12 @@ module.exports = function typeDetect(obj) {
   *   Int8Array          x 6,606,078 ops/sec ±1.74% (81 runs sampled)
   *   Uint8ClampedArray  x 6,602,224 ops/sec ±1.77% (83 runs sampled)
   */
-  var stringTag = (symbolToStringTagExists && obj[Symbol.toStringTag]);
+  const stringTag = (symbolToStringTagExists && obj[Symbol.toStringTag]);
   if (typeof stringTag === 'string') {
     return stringTag;
   }
 
-  var objPrototype = Object.getPrototypeOf(obj);
+  const objPrototype = Object.getPrototypeOf(obj);
   /* ! Speed optimisation
   * Pre:
   *   regex literal      x 1,772,385 ops/sec ±1.85% (77 runs sampled)
@@ -377,6 +375,4 @@ module.exports = function typeDetect(obj) {
     .toString
     .call(obj)
     .slice(toStringLeftSliceLength, toStringRightSliceLength);
-};
-
-module.exports.typeDetect = module.exports;
+}

karma.conf.js

@@ -1,14 +1,12 @@
-'use strict';
-
 /* eslint-disable no-process-env */
-var packageJson = require('./package.json');
-var defaultTimeout = 120000;
-var browserifyIstanbul = require('browserify-istanbul');
+const packageJson = require('./package.json');
+const defaultTimeout = 120000;
+const browserifyIstanbul = require('browserify-istanbul');
 module.exports = function configureKarma(config) {
-  var localBrowsers = [
+  const localBrowsers = [
     'PhantomJS',
   ];
-  var sauceLabsBrowsers = {
+  const sauceLabsBrowsers = {
     SauceChromeLatest: {
       base: 'SauceLabs',
       browserName: 'Chrome',
@@ -73,10 +71,10 @@ module.exports = function configureKarma(config) {
   });
 
   if (process.env.SAUCE_ACCESS_KEY && process.env.SAUCE_USERNAME) {
-    var branch = process.env.TRAVIS_BRANCH || 'local';
-    var build = 'localbuild';
+    const branch = process.env.TRAVIS_BRANCH || 'local';
+    let build = 'localbuild';
     if (process.env.TRAVIS_JOB_NUMBER) {
-      build = 'travis@' + process.env.TRAVIS_JOB_NUMBER;
+      build = `travis@${ process.env.TRAVIS_JOB_NUMBER }`;
     }
     config.reporters.push('saucelabs');
     config.set({
@@ -88,8 +86,8 @@ module.exports = function configureKarma(config) {
         recordVideo: true,
         startConnect: ('TRAVIS' in process.env) === false,
         tags: [
-          'typeDetect_' + packageJson.version,
-          process.env.SAUCE_USERNAME + '@' + branch,
+          `typeDetect_${ packageJson.version }`,
+          `${ process.env.SAUCE_USERNAME }@${ branch }`,
           build,
         ],
       },

package.json

@@ -40,14 +40,15 @@
       "es6": true
     },
     "extends": [
-      "strict/es5"
+      "strict/es6"
     ],
     "globals": {
       "HTMLElement": false
     },
     "rules": {
       "complexity": 0,
-      "max-statements": 0
+      "max-statements": 0,
+      "prefer-rest-params": 0
     }
   },
   "dependencies": {},