Replace JSCS and JSHint with ESLint

Author: eriwen

.eslintrc.yml

@@ -0,0 +1,38 @@
+env:
+  browser: true
+  commonjs: true
+  node: true
+extends: 'eslint:recommended'
+globals:
+  afterEach: false
+  beforeEach: false
+  define: false
+  describe: false
+  expect: false
+  it: false
+  jasmine: false
+  ES6Promise: false
+  Promise: false
+  SourceMap: false
+  StackFrame: false
+  StackTrace: false
+  StackTraceGPS: false
+parserOptions:
+  ecmaVersion: 5
+rules:
+  indent:
+    - error
+    - 4
+  linebreak-style:
+    - error
+    - unix
+  quotes:
+    - error
+    - single
+  semi:
+    - error
+    - always
+  no-console:
+    - error
+    - allow:
+      - error

.jscsrc

@@ -14,9 +14,9 @@ addons:
 services:
   - xvfb
 before_install:
-- export CHROME_BIN=chromium-browser
-- export DISPLAY=:99.0
+  - export CHROME_BIN=chromium-browser
+  - export DISPLAY=:99.0
 before_script:
-- npm install
+  - npm install
 script:
-- if [ "${TRAVIS_PULL_REQUEST}" = "false" ]; then gulp ci; else gulp pr; fi
+  - if [ "${TRAVIS_PULL_REQUEST}" = "false" ]; then npm run lint && npm run test-ci; else npm run test-pr; fi

.jshintignore

@@ -1,11 +1,7 @@
 var browserify = require('browserify');
 var concat = require('gulp-concat');
-var coveralls = require('gulp-coveralls');
 var del = require('del');
 var gulp = require('gulp');
-var jshint = require('gulp-jshint');
-var karma = require('karma');
-var path = require('path');
 var rename = require('gulp-rename');
 var runSequence = require('run-sequence');
 var sourcemaps = require('gulp-sourcemaps');
@@ -20,37 +16,6 @@ var polyfills = [
 ];
 var sources = 'stacktrace.js';
 
-gulp.task('lint', function() {
-    return gulp.src(sources)
-        .pipe(jshint())
-        .pipe(jshint.reporter('default'))
-        .pipe(jshint.reporter('fail'));
-});
-
-gulp.task('test', function(done) {
-    var server = new karma.Server({
-        configFile: __dirname + '/karma.conf.js',
-        singleRun: true
-    }, done);
-    server.start();
-});
-
-gulp.task('test-pr', ['dist'], function(done) {
-    new karma.Server({
-        configFile: __dirname + '/karma.conf.js',
-        browsers: ['Firefox', 'Chrome_Travis'],
-        singleRun: true
-    }, done).start();
-});
-
-gulp.task('test-ci', ['dist'], function(done) {
-    var server = new karma.Server({
-        configFile: __dirname + '/karma.conf.ci.js',
-        singleRun: true
-    }, done);
-    server.start();
-});
-
 gulp.task('dist', function() {
     browserify({
         entries: sources,
@@ -82,18 +47,8 @@ gulp.task('dist', function() {
         .pipe(gulp.dest('dist'));
 });
 
-gulp.task('clean', del.bind(null, ['build', 'coverage', 'dist']));
-
-gulp.task('pr', ['lint', 'test-pr'], function() {
-    gulp.src('./coverage/**/lcov.info')
-        .pipe(coveralls());
-});
-
-gulp.task('ci', ['lint', 'test-ci'], function() {
-    gulp.src('./coverage/**/lcov.info')
-        .pipe(coveralls());
-});
+gulp.task('clean', del.bind(null, ['build', 'dist']));
 
 gulp.task('default', ['clean'], function(cb) {
-    runSequence('lint', 'dist', 'test', cb);
+    runSequence('dist', 'test', cb);
 });

.jshintrc

@@ -5,45 +5,7 @@ module.exports = function(config) {
         process.exit(1);
     }
 
-    // Commented some of these out just so CI doesn't take forever.
-    // Check out https://saucelabs.com/platforms for all browser/platform combos
     var customLaunchers = {
-        slIOS9: {
-            appiumVersion: '1.6.3',
-            base: 'SauceLabs',
-            browserName: 'Safari',
-            deviceName: 'iPhone 6s Simulator',
-            deviceOrientation: 'portrait',
-            platformName: 'iOS',
-            platformVersion: '9.3'
-        },
-        slIOS10: {
-            appiumVersion: '1.6.3',
-            base: 'SauceLabs',
-            browserName: 'Safari',
-            deviceName: 'iPhone 7 Simulator',
-            deviceOrientation: 'portrait',
-            platformName: 'iOS',
-            platformVersion: '10.2'
-        },
-        slAndroid4: {
-            base: 'SauceLabs',
-            browserName: 'Android',
-            platform: 'Linux',
-            version: '4.4'
-        },
-        slAndroid5: {
-            base: 'SauceLabs',
-            browserName: 'Android',
-            platform: 'Linux',
-            version: '5.1'
-        },
-        slAndroid6: {
-            base: 'SauceLabs',
-            browserName: 'Android',
-            platform: 'Linux',
-            version: '6.0'
-        },
         slChrome: {
             base: 'SauceLabs',
             browserName: 'chrome',
@@ -59,35 +21,22 @@ module.exports = function(config) {
             browserName: 'firefox',
             version: 'latest'
         },
-        slSafari8: {
+        slFirefoxBeta: {
             base: 'SauceLabs',
-            browserName: 'safari',
-            platform: 'OS X 10.10',
-            version: '8.0'
-        },
-        slSafari9: {
-            base: 'SauceLabs',
-            browserName: 'safari',
-            platform: 'OS X 10.11',
-            version: '9.0'
+            browserName: 'firefox',
+            version: 'beta'
         },
-        slSafari10: {
+        slSafari: {
             base: 'SauceLabs',
             browserName: 'safari',
-            platform: 'macOS 10.12',
-            version: '10.0'
-        },
-        slEdge14: {
-            base: 'SauceLabs',
-            browserName: 'microsoftedge',
-            platform: 'Windows 10',
-            version: '14.14393'
+            platform: 'OS X 10.14',
+            version: 'latest'
         },
-        slEdge13: {
+        slEdge: {
             base: 'SauceLabs',
             browserName: 'microsoftedge',
             platform: 'Windows 10',
-            version: '13.10586'
+            version: 'latest'
         },
         slIE11: {
             base: 'SauceLabs',
@@ -98,28 +47,23 @@ module.exports = function(config) {
         slIE10: {
             base: 'SauceLabs',
             browserName: 'internet explorer',
-            platform: 'Windows 8',
+            platform: 'Windows 7',
             version: '10'
         },
         slIE9: {
             base: 'SauceLabs',
             browserName: 'internet explorer',
             platform: 'Windows 7',
-            version: '9.0'
+            version: '10',
+            'x-ua-compatible': 'IE=EmulateIE9'
+        },
+        slIE8: {
+            base: 'SauceLabs',
+            browserName: 'internet explorer',
+            platform: 'Windows 7',
+            version: '10',
+            'x-ua-compatible': 'IE=EmulateIE8'
         }
-        // Too unreliable on Sauce :(
-        // slIE8: {
-        //     base: 'SauceLabs',
-        //     browserName: 'internet explorer',
-        //     platform: 'Windows XP',
-        //     version: '8.0'
-        // }
-        //slIE7: {
-        //    base: 'SauceLabs',
-        //    browserName: 'internet explorer',
-        //    platform: 'Windows XP',
-        //    version: '7'
-        //}
     };
 
     config.set({
@@ -155,7 +99,7 @@ module.exports = function(config) {
         },
         customLaunchers: customLaunchers,
         browsers: Object.keys(customLaunchers),
-        reporters: ['progress', 'saucelabs', 'coverage'],
+        reporters: ['progress', 'saucelabs', 'coverage', 'coveralls'],
         preprocessors: {
             'stacktrace.js': 'coverage'
         },

.travis.yml

@@ -24,7 +24,7 @@ module.exports = function(config) {
             }
         },
         //browsers: ['ChromeCanary', 'Firefox'],
-        browsers: ['PhantomJS2'],
+        browsers: ['PhantomJS'],
         singleRun: false
     });
 };