ES6

* remove unused variables
* move `require`s at the top
* check for prototype built-ins
* run in strict mode
* always check for error values in tests
* make style consistent
* mark Node.js 6 as the minimum supported version in package.json

* Use the arrow return syntax

* logger.js: use `Boolean` to make the intention clearer.

* Use the rest params instead of arguments.

* test/getOptions.js: beautify git data.

* Update logger.js

Remove `hasDebugEnvVariable` function; it's just a `process.env` check

* Create .jshintrc

1 files changed, 13 insertions, 12 deletions

diff --git a/test/logger.js b/test/logger.js
index b007b6f..1d5261f 100644
--- a/test/logger.js
+++ b/test/logger.js
@@ -1,32 +1,33 @@
-var should = require('should');
-var sinon = require('sinon-restore');
-var index = require('../index');
+'use strict';
 
-describe("logger", function(){
-  it ("should log at debug level when --verbose is set", function(){
+require('should')();
+const index = require('..');
+
+describe('logger', () => {
+  it('should log at debug level when --verbose is set', () => {
     index.options.verbose = true;
-    var logger = require('../index').logger();
+    const logger = index.logger();
     logger.level.should.equal('debug');
   });
 
-  it ("should log at debug level when NODE_COVERALLS_DEBUG is set in env", function(){
+  it('should log at debug level when NODE_COVERALLS_DEBUG is set in env', () => {
     index.options.verbose = false;
     process.env.NODE_COVERALLS_DEBUG = 1;
-    var logger = require('../index').logger();
+    const logger = index.logger();
     logger.level.should.equal('debug');
   });
 
-  it ("should log at debug level when NODE_COVERALLS_DEBUG is set in env as a string", function(){
+  it('should log at debug level when NODE_COVERALLS_DEBUG is set in env as a string', () => {
     index.options.verbose = false;
     process.env.NODE_COVERALLS_DEBUG = '1';
-    var logger = require('../index').logger();
+    const logger = index.logger();
     logger.level.should.equal('debug');
   });
 
-  it ("should log at warn level when NODE_COVERALLS_DEBUG not set and no --verbose", function(){
+  it('should log at warn level when NODE_COVERALLS_DEBUG not set and no --verbose', () => {
     index.options.verbose = false;
     process.env.NODE_COVERALLS_DEBUG = 0;
-    var logger = require('../index').logger();
+    const logger = index.logger();
     logger.level.should.equal('error');
   });
 });