eslint-plugin-hammerhead - npm Package Compare versions

Comparing version 0.0.5 to 0.0.6

lib/index.js

@@ -6,8 +6,10 @@ "use strict";
         'proto-methods': require('./rules/proto-methods'),
-        'use-hh-promise': require('./rules/use-hh-promise')
+        'use-hh-promise': require('./rules/use-hh-promise'),
+        'use-native-methods': require('./rules/use-native-methods')
     },
     rulesConfig: {
         'proto-methods': 0,
-        'use-hh-promise': 0
+        'use-hh-promise': 0,
+        'use-native-methods': 0
     }
 };

package.json

 {
   "name": "eslint-plugin-hammerhead",
-  "version": "0.0.5",
+  "version": "0.0.6",
   "description": "Specific rules for the hammerhead.",
@@ -19,3 +19,3 @@ "keywords": [
   "devDependencies": {
-    "eslint": "~1.2.0"
+    "eslint": "^2.13.1"
   },
@@ -22,0 +22,0 @@ "engines": {

README.md

@@ -48,1 +48,2 @@ # eslint-plugin-hammerhead
 * use-hh-promise
+* use-native-methods

tests/lib/use-hh-promise.js

@@ -6,33 +6,29 @@ 'use strict';
 
-RuleTester.setDefaultConfig({
-    env:          { es6: true },
-    ecmaFeatures: { modules: true }
-});
-
-var ruleTester = new RuleTester();
-
-ruleTester.run('use-hh-promise', rule, {
+new RuleTester().run('use-hh-promise', rule, {
     valid: [
-        [
-            'var Promise = hammerhead.Promise;',
-            'var promise = new Promise(function(resolve, reject) {});'
-        ].join('\n'),
-        [
-            'let Promise = hammerhead.Promise;',
-            'Promise.all([',
-            '"something"',
-            ']);'
-        ].join('\n'),
-        [
-            'const Promise = hammerhead.Promise;',
-            'Promise.race([',
-            '"something"',
-            ']);'
-        ].join('\n'),
-        [
-            'import { Promise, something } from "deps/hammerhead";',
-            'Promise.race([',
-            '"something"',
-            ']);'
-        ].join('\n')
+        {
+            code: 'var Promise = hammerhead.Promise;' +
+                  'var promise = new Promise(function(resolve, reject) {});'
+        },
+        {
+            code: 'let Promise = hammerhead.Promise;' +
+                  'Promise.all([' +
+                  '"something"' +
+                  ']);',
+            parserOptions: { ecmaVersion: 6 }
+        },
+        {
+            code: 'const Promise = hammerhead.Promise;' +
+                  'Promise.race([' +
+                  '"something"' +
+                  ']);',
+            parserOptions: { ecmaVersion: 6 }
+        },
+        {
+            code: 'import { Promise, something } from "deps/hammerhead";' +
+                  'Promise.race([' +
+                  '"something"' +
+                  ']);',
+            parserOptions: { ecmaVersion: 6, sourceType: 'module' }
+        }
     ],
@@ -42,30 +38,22 @@
         {
-            code:   'var Promise = hammerhead.something;' +
-                    'var promise = new Promise(function(resolve, reject) {});',
-            errors: [
-                { message: rule.USE_HAMMERHEAD_PROMISE_ERR_MSG }
-            ]
+            code: 'var Promise = hammerhead.something;' +
+                  'var promise = new Promise(function(resolve, reject) {});',
+            errors: [{ message: rule.USE_HAMMERHEAD_PROMISE_ERR_MSG }]
         },
         {
-            code:   'var Promise = hh.Promise;' +
-                    'var promise = new Promise(function(resolve, reject) {});',
-            errors: [
-                { message: rule.USE_HAMMERHEAD_PROMISE_ERR_MSG }
-            ]
+            code: 'var Promise = hh.Promise;' +
+                  'var promise = new Promise(function(resolve, reject) {});',
+            errors: [{ message: rule.USE_HAMMERHEAD_PROMISE_ERR_MSG }]
         },
         {
-            code:   'var promise = new Promise(function(resolve, reject) {});',
-            errors: [
-                { message: rule.USE_HAMMERHEAD_PROMISE_ERR_MSG }
-            ]
+            code: 'var promise = new Promise(function(resolve, reject) {});',
+            errors: [{ message: rule.USE_HAMMERHEAD_PROMISE_ERR_MSG }]
         },
         {
-            code:   'Promise.all([' +
-                    '"something"' +
-                    ']);',
-            errors: [
-                { message: rule.USE_HAMMERHEAD_PROMISE_ERR_MSG }
-            ]
+            code: 'Promise.all([' +
+                  '"something"' +
+                  ']);',
+            errors: [{ message: rule.USE_HAMMERHEAD_PROMISE_ERR_MSG }]
         }
     ]
 });

New alerts

Dynamic require

Supply chain risk

Dynamic require can indicate the package is performing dangerous or unsafe dynamic code execution.

Found 1 instance in 1 package

Filesystem access

Supply chain risk

Accesses the file system, and could potentially read sensitive data.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

18150

increased by93.6%

Number of package files

11

increased by37.5%

Lines of code

368

increased by88.72%

Number of lines in readme file

49

increased by2.08%

Worsened metrics

Number of low supply chain risk alerts

3

increased by200%

No dependency changes