eslint-config-wordpress - npm Package Compare versions

Comparing version 0.1.0 to 1.0.0

CHANGELOG.md

@@ -0,3 +1,8 @@
+# 1.0.0
+
+* fix: Use `eslint-plugin-wordpress/jshint` shared configuration.
+* chore: Removed NodeJS 0.10.x support, `eslint` and `eslint-config-wordpress` now require NodeJS > 4.2.1 LTS or greater
+
 # 0.1.0
 
 * Initial release

index.js

 module.exports = {
 
+	env: {
+		browser: true
+	},
+
 	globals: {
-		MyGlobal: true
+		_: false,
+		Backbone: false,
+		jQuery: false,
+		JSON: false,
+		wp: false
 	},
@@ -9,162 +17,30 @@
 		// Possible Errors
-		'comma-dangle': 2,
-		'no-cond-assign': 2,
-		'no-console': 2,
-		'no-constant-condition': 2,
-		'no-control-regex': 2,
-		'no-debugger': 2,
-		'no-dupe-args': 2,
-		'no-dupe-keys': 2,
-		'no-duplicate-case': 2,
-		'no-empty-character-class': 2,
-		'no-empty': 2,
-		'no-ex-assign': 2,
-		'no-extra-boolean-cast': 2,
-		'no-extra-parens': 2,
-		'no-extra-semi': 2,
-		'no-func-assign': 2,
-		'no-inner-declarations': 2,
-		'no-invalid-regexp': 2,
-		'no-irregular-whitespace': 2,
-		'no-negated-in-lhs': 2,
-		'no-obj-calls': 2,
-		'no-regex-spaces': 2,
-		'no-sparse-arrays': 2,
-		'no-unreachable': 2,
-		'use-isnan': 2,
-		'valid-jsdoc': 2,
-		'valid-typeof': 2,
-		'no-unexpected-multiline': 2,
+		// Disallow assignment in conditional expressions
+		'no-cond-assign': ['error', 'except-parens'],
+		// Disallow irregular whitespace outside of strings and comments
+		'no-irregular-whitespace': 'error',
 		// Best Practices
-		'accessor-pairs': 2,
-		'block-scoped-var': 2,
-		'complexity': 2,
-		'consistent-return': 2,
-		'curly': 2,
-		'default-case': 2,
-		'dot-notation': 2,
-		'dot-location': 2,
-		'eqeqeq': 2,
-		'guard-for-in': 2,
-		'no-alert': 2,
-		'no-caller': 2,
-		'no-div-regex': 2,
-		'no-else-return': 2,
-		'no-empty-label': 2,
-		'no-eq-null': 2,
-		'no-eval': 2,
-		'no-extend-native': 2,
-		'no-extra-bind': 2,
-		'no-fallthrough': 2,
-		'no-floating-decimal': 2,
-		'no-implicit-coercion': 2,
-		'no-implied-eval': 2,
-		'no-invalid-this': 2,
-		'no-iterator': 2,
-		'no-labels': 2,
-		'no-lone-blocks': 2,
-		'no-loop-func': 2,
-		'no-multi-spaces': 2,
-		'no-multi-str': 2,
-		'no-native-reassign': 2,
-		'no-new-func': 2,
-		'no-new-wrappers': 2,
-		'no-new': 2,
-		'no-octal-escape': 2,
-		'no-octal': 2,
-		'no-param-reassign': 2,
-		'no-process-env': 2,
-		'no-proto': 2,
-		'no-redeclare': 2,
-		'no-return-assign': 2,
-		'no-script-url': 2,
-		'no-self-compare': 2,
-		'no-sequences': 2,
-		'no-throw-literal': 2,
-		'no-unused-expressions': 2,
-		'no-useless-call': 2,
-		'no-useless-concat': 2,
-		'no-void': 2,
-		'no-warning-comments': 2,
-		'no-with': 2,
-		'radix': 2,
-		'vars-on-top': 2,
-		'wrap-iife': 2,
-		'yoda': 2,
+		// Specify curly brace conventions for all control statements
+		'curly': 'error',
+		// Require the use of === and !==
+		'eqeqeq': 'error',
+		// Disallow use of arguments.caller or arguments.callee
+		'no-caller': 'error',
+		// Disallow comparisons to null without a type-checking operator
+		'no-eq-null': 'error',
+		// Disallow usage of expressions in statement position
+		'no-unused-expressions': 'error',
+		// Require immediate function invocation to be wrapped in parentheses
+		'wrap-iife': ['error', 'any'],
 		// Strict Mode
-		'strict': 2,
 		// Variables
-		'init-declarations': 2,
-		'no-catch-shadow': 2,
-		'no-delete-var': 2,
-		'no-label-var': 2,
-		'no-shadow-restricted-names': 2,
-		'no-shadow': 2,
-		'no-undef-init': 2,
-		'no-undef': 2,
-		'no-undefined': 2,
-		'no-unused-vars': 2,
-		'no-use-before-define': 2,
+		// Disallow use of undeclared variables unless mentioned in a /*global */ block
+		'no-undef': 'error',
+		// Disallow declaration of variables that are not used in the code
+		'no-unused-vars': 'error',
 		// Stylistic Issues
-		'array-bracket-spacing': [2, 'always'],
-		'block-spacing': 2,
-		'brace-style': 2,
-		'camelcase': 2,
-		'comma-spacing': [2, {before: false, after: true}],
-		'comma-style': [2, 'last'],
-		'computed-property-spacing': 2,
-		'consistent-this': 2,
-		'eol-last': 2,
-		'func-names': 2,
-		'func-style': 2,
-		'id-length': 2,
-		'id-match': 2,
-		'indent': [2, 'tab', {SwitchCase: 1}],
-		'key-spacing': 2,
-		'lines-around-comment': 2,
-		'linebreak-style': 2,
-		'max-nested-callbacks': 2,
-		'new-cap': 2,
-		'new-parens': 2,
-		'newline-after-var': 2,
-		'no-array-constructor': 2,
-		'no-continue': 2,
-		'no-inline-comments': 2,
-		'no-lonely-if': 2,
-		'no-mixed-spaces-and-tabs': 2,
-		'no-multiple-empty-lines': 2,
-		'no-nested-ternary': 2,
-		'no-new-object': 2,
-		'no-spaced-func': 2,
-		'no-ternary': 2,
-		'no-trailing-spaces': 2,
-		'no-underscore-dangle': 2,
-		'no-unneeded-ternary': 2,
-		'object-curly-spacing': 2,
-		'one-var': 2,
-		'operator-assignment': 2,
-		'operator-linebreak': 2,
-		'padded-blocks': [2, 'never'],
-		'quote-props': 2,
-		'quotes': [2, 'single'],
-		'semi-spacing': 2,
-		'semi': [2, 'always'],
-		'sort-vars': 2,
-		'space-after-keywords': 2,
-		'space-before-blocks': 2,
-		'space-before-function-paren': 2,
-		'space-in-parens': [2, 'always'],
-		'space-infix-ops': 2,
-		'space-return-throw-case': 2,
-		'space-unary-ops': 2,
-		'spaced-comment': 2,
-		'wrap-regex': 2,
+		// Specify whether backticks, double or single quotes should be used
+		'quotes': ['error', 'single']
 		// Legacy
-		'max-depth': 2,
-		'max-len': [2, 100, 4],
-		'max-params': 2,
-		'max-statements': 2,
-		'no-bitwise': 2,
-		'no-plusplus': 2
 	}
 };

package.json

 {
   "name": "eslint-config-wordpress",
-  "version": "0.1.0",
+  "version": "1.0.0",
   "description": "ESLint shareable config for WordPress",
   "license": "MIT",
   "repository": "ntwb/eslint-config-wordpress",
+  "homepage": "https://github.com/ntwb/eslint-config-wordpress#readme",
   "author": {
     "name": "Stephen Edgar",
     "email": "[e-mail]",
-    "url": "netweb.com.au"
-    },
+    "url": "https://netweb.com.au"
+  },
   "engines": {
-    "node": ">=0.10.0"
+    "node": ">=4.2.1"
   },
@@ -20,3 +21,4 @@ "main": "index.js",
   "scripts": {
-    "test": "test"
+    "release": "npmpub --verbose",
+    "test": "ava --verbose"
   },
@@ -26,2 +28,3 @@ "keywords": [
     "eslintconfig",
+    "eslint-config",
     "eslint-config-wordpress",
@@ -36,4 +39,7 @@ "JavaScript",
   ],
-
-  "homepage": "https://github.com/ntwb/eslint-config-wordpress#readme"
+  "devDependencies": {
+    "ava": "^0.16.0",
+    "is-plain-obj": "^1.1.0",
+    "npmpub": "^3.1.0"
+  }
 }

README.md

 # eslint-config-wordpress
 
-> WordPress JavaScript Coding Standards shareable config for ESLint.
+[WordPress JavaScript Coding Standards](https://make.wordpress.org/core/handbook/best-practices/coding-standards/javascript/) shareable config for ESLint.
 
-WordPress JavaScript Coding Standards
-https://make.wordpress.org/core/handbook/best-practices/coding-standards/javascript/
+[![Build Status](https://api.travis-ci.org/ntwb/eslint-config-wordpress.svg?branch=master)](https://travis-ci.org/ntwb/eslint-config-wordpress) [![bitHound Overall Score](https://www.bithound.io/github/ntwb/eslint-config-wordpress/badges/score.svg)](https://www.bithound.io/github/ntwb/eslint-config-wordpress) [![NPM version](http://img.shields.io/npm/v/eslint-config-wordpress.svg)](https://www.npmjs.org/package/eslint-config-wordpress)
 
@@ -8,0 +7,0 @@ ## Installation

New alerts

Major refactor

Supply chain risk

Package has recently undergone a major refactor. It may be unstable or indicate significant internal changes. Use caution when updating to versions that include significant changes.

Found 1 instance in 1 package

Fixed alerts

No v1

Quality

Package is not semver >=1. This means it is not stable and does not support ^ ranges.

Found 1 instance in 1 package

Improved metrics

Number of low quality alerts

0

decreased by-100%

Worsened metrics

Total package byte prevSize

4260

decreased by-28.98%

Dev dependency count

3

increased byInfinity%

Lines of code

42

decreased by-74.85%

Number of lines in readme file

22

decreased by-4.35%

Number of medium supply chain risk alerts

1

increased byInfinity%

No dependency changes