Merge pull request #6 from codeclimate/abh-upstream-v1.10.3

Abh upstream v1.10.3

Author: Ashley Baldwin-Hunter

.eslintrc renamed to .eslintrc.yml

@@ -1,3 +1,75 @@
+v1.10.3 - December 1, 2015
+
+* Docs: Update strict rule docs (fixes #4583) (Nicholas C. Zakas)
+* Docs: Reference .eslintrc.* in contributing docs (fixes #4532) (Kai Cataldo)
+* Fix: Add for-of to `curly` rule (fixes #4571) (Kai Cataldo)
+* Fix: Ignore space before function in array start (fixes #4569) (alberto)
+
+v1.10.2 - November 27, 2015
+
+* Upgrade: [email protected] (refs #4485) (Nicholas C. Zakas)
+* Upgrade: Pinned down js-yaml to avoid breaking dep (fixes #4553) (alberto)
+* Fix: lines-around-comment with multiple comments (fixes #3509) (alberto)
+* Upgrade: [email protected] (fixes #4545) (Kevin Partington)
+* Fix: Bugfix for eqeqeq autofix (fixes #4540) (Kevin Partington)
+* Fix: Add for-in to `curly` rule (fixes #4436) (Kai Cataldo)
+* Fix: `valid-jsdoc` unneeded require check fix (fixes #4527) (Gyandeep Singh)
+* Fix: `brace-style` ASI fix for if-else condition (fixes #4520) (Gyandeep Singh)
+* Build: Add branch update during release process (fixes #4491) (Gyandeep Singh)
+* Build: Allow revert commits in commit messages (fixes #4452) (alberto)
+* Fix: Incorrect location in no-fallthrough (fixes #4516) (alberto)
+* Fix: `no-spaced-func` had been crashed (fixes #4508) (Toru Nagashima)
+* Fix: Add a RestProperty test of `no-undef` (fixes #3271) (Toru Nagashima)
+* Docs: Load badge from HTTPS (Brian J Brennan)
+* Build: Update eslint bot messages (fixes #4497) (Nicholas C. Zakas)
+
+v1.10.1 - November 20, 2015
+
+* Fix: Revert freezing context object (refs #4495) (Nicholas C. Zakas)
+* 1.10.0 (Nicholas C. Zakas)
+
+v1.10.0 - November 20, 2015
+
+* Docs: Remove dupes from changelog (Nicholas C. Zakas)
+* Update: --init to create extensioned files (fixes #4476) (Nicholas C. Zakas)
+* Docs: Update description of exported comment (fixes #3916) (Nicholas C. Zakas)
+* Docs: Move legacy rules to stylistic (files #4111) (Nicholas C. Zakas)
+* Docs: Clean up description of recommended rules (fixes #4365) (Nicholas C. Zakas)
+* Docs: Fix home directory config description (fixes #4398) (Nicholas C. Zakas)
+* Update: Add class support to `require-jsdoc` rule (fixes #4268) (Gyandeep Singh)
+* Update: return type error in `valid-jsdoc` rule (fixes #4443) (Gyandeep Singh)
+* Update: Display errors at the place where fix should go (fixes #4470) (nightwing)
+* Docs: Fix typo in default `cacheLocation` value (Andrew Hutchings)
+* Fix: Handle comments in block-spacing (fixes #4387) (alberto)
+* Update: Accept array for `ignorePattern` (fixes #3982) (Jesse McCarthy)
+* Update: replace label and break with IIFE and return (fixes #4459) (Ilya Panasenko)
+* Fix: space-before-keywords false positive (fixes #4449) (alberto)
+* Fix: Improves performance (refs #3530) (Toru Nagashima)
+* Fix: Autofix quotes produces invalid javascript (fixes #4380) (nightwing)
+* Docs: Update indent.md (Nathan Brown)
+* New: Disable comment config option (fixes #3901) (Matthew Riley MacPherson)
+* New: Config files with extensions (fixes #4045, fixes #4263) (Nicholas C. Zakas)
+* Revert "Update: Add JSX exceptions to no-extra-parens (fixes #4229)" (Brandon Mills)
+* Update: Add JSX exceptions to no-extra-parens (fixes #4229) (Brandon Mills)
+* Docs: Replace link to deprecated rule with newer rule (Andrew Marshall)
+* Fix: `no-extend-native` crashed at empty defineProperty (fixes #4438) (Toru Nagashima)
+* Fix: Support empty if blocks in lines-around-comment (fixes #4339) (alberto)
+* Fix: `curly` warns wrong location for `else` (fixes #4362) (Toru Nagashima)
+* Fix: `id-length` properties never option (fixes #4347) (Toru Nagashima)
+* Docs: missing close rbracket in example (@storkme)
+* Revert "Update: Allow empty arrow body (fixes #4411)" (Nicholas C. Zakas)
+* Fix: eqeqeq autofix avoids clashes with space-infix-ops (fixes #4423) (Kevin Partington)
+* Docs: Document semi-spacing behaviour (fixes #4404) (alberto)
+* Update: Allow empty arrow body (fixes #4411) (alberto)
+* Fix: Handle comments in comma-spacing (fixes #4389) (alberto)
+* Update: Refactor eslint.verify args (fixes #4395) (Nicholas C. Zakas)
+* Fix: no-undef-init should ignore const (fixes #4284) (Nicholas C. Zakas)
+* Fix: Add the missing "as-needed" docs to the radix rule (fixes #4364) (Michał Gołębiowski)
+* Fix: Display singular/plural version of "line" in message (fixes #4359) (Marius Schulz)
+* Update: Add Popular Style Guides (fixes #4320) (Jamund Ferguson)
+* Fix: eslint.report can be called w/o node if loc provided (fixes #4220) (Kevin Partington)
+* Update: no-implicit-coercion validate AssignmentExpression (fixes #4348) (Ilya Panasenko)
+
 v1.9.0 - November 6, 2015
 
 * Update: Make radix accept a "as-needed" option (fixes #4048) (Michał Gołębiowski)

CHANGELOG.md

@@ -182,6 +182,9 @@ function getReleaseType(version) {
 function release(type) {
     var newVersion;/* , changes;*/
 
+    exec("git checkout master && git fetch origin && git reset --hard origin/master");
+    exec("npm install && npm prune");
+
     target.test();
     echo("Generating new version");
     newVersion = execSilent("npm version " + type).trim();
@@ -944,7 +947,7 @@ target.checkGitCommit = function() {
     }
 
     // Only check non-release messages
-    if (!semver.valid(commitMsgs[0])) {
+    if (!semver.valid(commitMsgs[0]) && !/^Revert /.test(commitMsgs[0])) {
         if (commitMsgs[0].slice(0, commitMsgs[0].indexOf("\n")).length > 72) {
             echo(" - First line of commit message must not exceed 72 characters");
             failed = true;

Makefile.js

@@ -122,5 +122,5 @@ Join our [Mailing List](https://groups.google.com/group/eslint) or [Chatroom](ht
 [travis-url]: https://travis-ci.org/eslint/eslint
 [coveralls-image]: https://img.shields.io/coveralls/eslint/eslint/master.svg?style=flat-square
 [coveralls-url]: https://coveralls.io/r/eslint/eslint?branch=master
-[downloads-image]: http://img.shields.io/npm/dm/eslint.svg?style=flat-square
+[downloads-image]: https://img.shields.io/npm/dm/eslint.svg?style=flat-square
 [downloads-url]: https://www.npmjs.com/package/eslint

README.md

@@ -46,14 +46,13 @@ if (useStdIn) {
         }
     }));
 } else if (init) {
-    var configInit = require("../lib/config-initializer");
+    var configInit = require("../lib/config/config-initializer");
     configInit.initializeConfig(function(err) {
         if (err) {
             exitCode = 1;
             console.error(err.message);
             console.error(err.stack);
         } else {
-            console.log("Successfully created .eslintrc file in " + process.cwd());
             exitCode = 0;
         }
     });

bin/eslint.js

@@ -28,8 +28,10 @@ The most important method on `linter` is `verify()`, which initiates linting of
 
 * `code` - the source code to lint (a string or instance of `SourceCode`).
 * `config` - a configuration object.
-* `filename` - (optional) the filename to associate with the source code.
-* `saveState` - (optional) set to true to maintain the internal state of `linter` after linting (mostly used for testing purposes).
+* `options` - (optional) Additional options for this run.
+    * `filename` - (optional) the filename to associate with the source code.
+    * `saveState` - (optional) set to true to maintain the internal state of `linter` after linting (mostly used for testing purposes).
+    * `allowInlineConfig` - (optional) set to `false` to disable inline comments from changing eslint rules.
 
 You can call `verify()` like this:
 
@@ -40,7 +42,7 @@ var messages = linter.verify("var foo;", {
     rules: {
         semi: 2
     }
-}, "foo.js");
+}, { filename: "foo.js" });
 
 // or using SourceCode
 
@@ -53,7 +55,7 @@ var messages = linter.verify(code, {
     rules: {
         semi: 2
     }
-}, "foo.js");
+}, { filename: "foo.js" });
 ```
 
 The `verify()` method returns an array of objects containing information about the linting warnings and errors. Here's an example:
@@ -95,7 +97,7 @@ var messages = linter.verify("var foo = bar;", {
     rules: {
         semi: 2
     }
-}, "foo.js");
+}, { filename: "foo.js" });
 
 var code = linter.getSourceCode();
 
@@ -123,14 +125,15 @@ The `CLIEngine` is a constructor, and you can create a new instance by passing i
 * `fix` - True indicates that fixes should be applied to the text when possible.
 * `ignore` - False disables use of `.eslintignore` (default: true). Corresponds to `--no-ignore`.
 * `ignorePath` - The ignore file to use instead of `.eslintignore` (default: null). Corresponds to `--ignore-path`.
+* `ignorePattern` - Glob patterns for paths to ignore. String or array of strings.
 * `baseConfig` - Set to false to disable use of base config. Could be set to an object to override default base config as well.
 * `rulePaths` - An array of directories to load custom rules from (default: empty array). Corresponds to `--rulesdir`.
 * `rules` - An object of rules to use (default: null). Corresponds to `--rule`.
 * `useEslintrc` - Set to false to disable use of `.eslintrc` files (default: true). Corresponds to `--no-eslintrc`.
 * `parser` - Specify the parser to be used (default: `espree`). Corresponds to `--parser`.
 * `cache` - Operate only on changed files (default: `false`). Corresponds to `--cache`.
 * `cacheFile` - Name of the file where the cache will be stored (default: `.eslintcache`). Corresponds to `--cache-file`. Deprecated: use `cacheLocation` instead.
-* `cacheLocation` - Name of the file or directory where the cache will be stored (default: `.estlintcache`). Correspond to `--cache-location`
+* `cacheLocation` - Name of the file or directory where the cache will be stored (default: `.eslintcache`). Correspond to `--cache-location`
 
 For example:
 

docs/developer-guide/nodejs-api.md

@@ -118,6 +118,8 @@ The main method you'll use is `context.report()`, which publishes a warning or e
 * `data` - (optional) placeholder data for `message`.
 * `fix` - (optional) a function that applies a fix to resolve the problem.
 
+Note that at least one of `node` or `loc` is required.
+
 The simplest example is to use just `node` and `message`:
 
 ```js

docs/developer-guide/working-with-rules.md

@@ -1,6 +1,6 @@
 # Rules
 
-Rules in ESLint are divided into several categories to help you better understand their value. Though none are enabled by default, you can turn on rules that ESLint recommends by specifying your configuration to inherit from `eslint:recommended`. The rules that will be enabled when you inherit from `eslint:recommended` are indicated below as "(recommended)". For more information on how to configure rules and inherit from `eslint:recommended`, please see the [configuration documentation](../user-guide/configuring.md).
+Rules in ESLint are divided into several categories to help you better understand their value. All rules are disabled by default. ESLint recommends some rules to catch common problems, and you can use these recommended rules by including `extends: "eslint:recommended"` in your configuration file. The rules that will be enabled when you inherit from `eslint:recommended` are indicated below as "(recommended)". For more information on how to configure rules and use `extends`, please see the [configuration documentation](../user-guide/configuring.md).
 
 Some rules are fixable using the `--fix` command line flag. Those rules are marked as "(fixable)" below.
 
@@ -158,11 +158,16 @@ These rules are purely matters of style and are quite subjective.
 * [key-spacing](key-spacing.md) - enforce spacing between keys and values in object literal properties
 * [linebreak-style](linebreak-style.md) - disallow mixed 'LF' and 'CRLF' as linebreaks
 * [lines-around-comment](lines-around-comment.md) - enforce empty lines around comments
+* [max-depth](max-depth.md) - specify the maximum depth that blocks can be nested
+* [max-len](max-len.md) - specify the maximum length of a line in your program
 * [max-nested-callbacks](max-nested-callbacks.md) - specify the maximum depth callbacks can be nested
+* [max-params](max-params.md) - limits the number of parameters that can be used in the function declaration.
+* [max-statements](max-statements.md) - specify the maximum number of statement allowed in a function
 * [new-cap](new-cap.md) - require a capital letter for constructors
 * [new-parens](new-parens.md) - disallow the omission of parentheses when invoking a constructor with no arguments
 * [newline-after-var](newline-after-var.md) - require or disallow an empty newline after variable declarations
 * [no-array-constructor](no-array-constructor.md) - disallow use of the `Array` constructor
+* [no-bitwise](no-bitwise.md) - disallow use of bitwise operators
 * [no-continue](no-continue.md) - disallow use of the `continue` statement
 * [no-inline-comments](no-inline-comments.md) - disallow comments inline after code
 * [no-lonely-if](no-lonely-if.md) - disallow `if` as the only statement in an `else` block
@@ -171,6 +176,7 @@ These rules are purely matters of style and are quite subjective.
 * [no-negated-condition](no-negated-condition.md) - disallow negated conditions
 * [no-nested-ternary](no-nested-ternary.md) - disallow nested ternary expressions
 * [no-new-object](no-new-object.md) - disallow the use of the `Object` constructor
+* [no-plusplus](no-plusplus.md) - disallow use of unary operators, `++` and `--`
 * [no-restricted-syntax](no-restricted-syntax.md) - disallow use of certain syntax in code
 * [no-spaced-func](no-spaced-func.md) - disallow space between function identifier and application (fixable)
 * [no-ternary](no-ternary.md) - disallow the use of ternary operators
@@ -222,16 +228,6 @@ These rules are only relevant to ES6 environments.
 * [prefer-template](prefer-template.md) - suggest using template literals instead of strings concatenation
 * [require-yield](require-yield.md) - disallow generator functions that do not have `yield`
 
-## Legacy
-
-The following rules are included for compatibility with [JSHint](http://jshint.com/) and [JSLint](http://jslint.com/). While the names of the rules may not match up with the JSHint/JSLint counterpart, the functionality is the same.
-
-* [max-depth](max-depth.md) - specify the maximum depth that blocks can be nested
-* [max-len](max-len.md) - specify the maximum length of a line in your program
-* [max-params](max-params.md) - limits the number of parameters that can be used in the function declaration.
-* [max-statements](max-statements.md) - specify the maximum number of statement allowed in a function
-* [no-bitwise](no-bitwise.md) - disallow use of bitwise operators
-* [no-plusplus](no-plusplus.md) - disallow use of unary operators, `++` and `--`
 
 ## Removed
 

docs/rules/README.md

@@ -46,7 +46,7 @@ Level of indentation denotes the multiple of the indent specified. Example:
 * Indent of tabs with SwitchCase set to 2 will indent `SwitchCase` with 2 tabs with respect to switch.
 
 
-2 space indentation with enabled switch cases validation
+2 space indentation with enabled switch cases indentation
 
 ```json
  "indent": [2, 2, {"SwitchCase": 1}]

docs/rules/indent.md

@@ -11,6 +11,7 @@ var b = ~foo.indexOf(".");
 var n = +foo;
 var n = 1 * foo;
 var s = "" + foo;
+foo += "";
 ```
 
 Those can be replaced with the following code:
@@ -21,6 +22,7 @@ var b = foo.indexOf(".") !== -1;
 var n = Number(foo);
 var n = Number(foo);
 var s = String(foo);
+foo = String(foo);
 ```
 
 ## Rule Details
@@ -96,6 +98,8 @@ The following patterns are considered problems:
 /*eslint no-implicit-coercion: 2*/
 
 var n = "" + foo; /*error use `String(foo)` instead.*/
+
+foo += ""; /*error use `foo = String(foo)` instead.*/
 ```
 
 The following patterns are not considered problems: