eslint-config-hapi - npm Package Compare versions

Comparing version 8.0.1 to 9.0.0

lib/index.js

@@ -14,3 +14,3 @@ 'use strict';
     'no-console': 0,
-    'no-constant-condition': 0,
+    'no-constant-condition': 2,
     'no-empty': 0,
@@ -50,11 +50,10 @@ 'no-native-reassign': 0,
     'no-array-constructor': 2,
-    'no-arrow-condition': 2,
+    'no-confusing-arrow': 2,
     'no-new-object': 2,
     'no-spaced-func': 2,
     'no-mixed-spaces-and-tabs': 2,
-    'space-after-keywords': 2,
+    'keyword-spacing': [2, { 'before': false, 'after': true }],
     'semi': [2, 'always'],
     'semi-spacing': [2, { 'before': false, 'after': true }],
     'space-infix-ops': 2,
-    'space-return-throw-case': 2,
     'space-unary-ops': [1, { 'words': true, 'nonwords': false }],
@@ -61,0 +60,0 @@ 'strict': [2, 'global'],

package.json

 {
   "name": "eslint-config-hapi",
-  "version": "8.0.1",
+  "version": "9.0.0",
   "description": "Shareable ESLint config for the hapi ecosystem",
@@ -26,3 +26,3 @@ "author": "Continuation Labs <[e-mail]> (http://continuation.io/)",
     "code": "1.x.x",
-    "eslint": "1.x.x",
+    "eslint": "2.x.x",
     "eslint-plugin-hapi": "4.x.x",
@@ -29,0 +29,0 @@ "lab": "6.x.x"

test/fixtures/brace-style.js

@@ -1,2 +0,2 @@
-/* eslint-disable strict */
+/* eslint-disable strict, keyword-spacing */
 const foo = true;
@@ -3,0 +3,0 @@ let bar = 0;

test/fixtures/one-var.js

@@ -1,2 +0,2 @@
-/* eslint-disable no-unused-vars */
+/* eslint-disable no-unused-vars, prefer-const */
 'use strict';
@@ -3,0 +3,0 @@ const foo = 1;

test/index.js

@@ -47,3 +47,3 @@ 'use strict';
     expect(msg.severity).to.equal(2);
-    expect(msg.message).to.equal('Use the global form of "use strict".');
+    expect(msg.message).to.equal('Use the global form of \'use strict\'.');
     expect(msg.line).to.equal(2);
@@ -319,3 +319,3 @@ expect(msg.column).to.equal(1);
     expect(msg.severity).to.equal(1);
-    expect(msg.message).to.equal('"res" is already declared in the upper scope.');
+    expect(msg.message).to.equal('\'res\' is already declared in the upper scope.');
     expect(msg.line).to.equal(27);
@@ -362,3 +362,3 @@ expect(msg.column).to.equal(33);
     expect(msg.severity).to.equal(1);
-    expect(msg.message).to.equal('"internals2" is defined but never used');
+    expect(msg.message).to.equal('\'internals2\' is defined but never used');
     expect(msg.line).to.equal(3);
@@ -384,3 +384,3 @@ expect(msg.column).to.equal(7);
     expect(msg.severity).to.equal(2);
-    expect(msg.message).to.equal('`foo` is never modified, use `const` instead.');
+    expect(msg.message).to.equal('\'foo\' is never modified, use \'const\' instead.');
     expect(msg.line).to.equal(4);
@@ -485,4 +485,4 @@ expect(msg.column).to.equal(5);
 
-  it('enforces no-arrow-condition rule', function (done) {
-    var output = lintFile('fixtures/no-arrow-condition.js');
+  it('enforces no-constant-condition rule', function (done) {
+    var output = lintFile('fixtures/no-constant-condition.js');
     var results = output.results[0];
@@ -497,5 +497,5 @@
 
-    expect(msg.ruleId).to.equal('no-arrow-condition');
+    expect(msg.ruleId).to.equal('no-constant-condition');
     expect(msg.severity).to.equal(2);
-    expect(msg.message).to.equal('Arrow function `=>` used inside IfStatement instead of comparison operator.');
+    expect(msg.message).to.equal('Unexpected constant condition.');
     expect(msg.line).to.equal(3);
@@ -502,0 +502,0 @@ expect(msg.column).to.equal(1);

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

31399

increased by0.01%

Worsened metrics

Lines of code

769

decreased by-0.13%

No dependency changes