Merge branch 'master' into benchmark

# Conflicts:
#	package.json

Author: Manuel Lopez

CHANGELOG.md

@@ -1,5 +1,9 @@
 # Change Log
 
+## [v4.0.0] - April 4, 2017
+### Upgraded to [TSLint 5.0](https://github.com/palantir/tslint/releases/tag/5.0.0)
+
+
 ## [v3.5.1] - 2017/03/17 23:00 GMT-0600
 #### Fixes
 - sort-imports bug in simple type imports ([3d57149])
@@ -274,6 +278,7 @@ Note that the npm release `v3.2.1` only contains this hotfix [cb01358].
 - Added and made lots of improvements in README.md
 
 
+[v4.0.0]: https://github.com/buzinas/tslint-eslint-rules/compare/v3.5.1...v4.0.0
 [v3.5.1]: https://github.com/buzinas/tslint-eslint-rules/compare/v3.5.0...v3.5.1
 [v3.5.0]: https://github.com/buzinas/tslint-eslint-rules/compare/v3.4.0...v3.5.0
 [v3.4.0]: https://github.com/buzinas/tslint-eslint-rules/compare/v3.3.0...v3.4.0

CONTRIBUTING.md

@@ -18,12 +18,10 @@ the issue easier and quicker.
 
  * tslint and typescript version.
 
-       ```sh
-$ tslint --version
-4.0.0
-$ tsc --version
-Version 2.0.10
-       ```
+       $ tslint --version
+       4.5.1
+       $ tsc --version
+       Version 2.2.1
 
  * `tslint.json` configuration.
  * typescript code being linted.

README.md

@@ -84,11 +84,11 @@ The following rules point out areas where you might have made mistakes.
 |:no_entry_sign:|[no-func-assign](http://eslint.org/docs/rules/no-func-assign)|Not applicable|disallow overwriting functions written as function declarations (recommended)|
 |:white_check_mark:|[no-inner-declarations](http://eslint.org/docs/rules/no-inner-declarations)|[no-inner-declarations](https://github.com/buzinas/tslint-eslint-rules/blob/master/src/docs/rules/noInnerDeclarationsRule.md)|disallow function or variable declarations in nested blocks (recommended)|
 |:white_check_mark:|[no-invalid-regexp](http://eslint.org/docs/rules/no-invalid-regexp)|[no-invalid-regexp](https://github.com/buzinas/tslint-eslint-rules/blob/master/src/docs/rules/noInvalidRegexpRule.md)|disallow invalid regular expression strings in the `RegExp` constructor (recommended)|
-|:white_check_mark:|[no-irregular-whitespace](http://eslint.org/docs/rules/no-irregular-whitespace)|[no-irregular-whitespace](https://github.com/buzinas/tslint-eslint-rules/blob/master/src/docs/rules/noIrregularWhitespaceRule.md)|disallow irregular whitespace outside of strings and comments (recommended)|
+|:white_check_mark:|[no-irregular-whitespace](http://eslint.org/docs/rules/no-irregular-whitespace)|[ter-no-irregular-whitespace](https://github.com/buzinas/tslint-eslint-rules/blob/master/src/docs/rules/terNoIrregularWhitespaceRule.md)|disallow irregular whitespace (recommended)|
 |:no_entry_sign:|[no-negated-in-lhs](http://eslint.org/docs/rules/no-negated-in-lhs)|Not applicable|disallow negation of the left operand of an `in` expression (recommended)|
 |:no_entry_sign:|[no-obj-calls](http://eslint.org/docs/rules/no-obj-calls)|Not applicable|disallow the use of object properties of the global object (`Math` and `JSON`) as functions (recommended)|
 |:white_check_mark:|[no-regex-spaces](http://eslint.org/docs/rules/no-regex-spaces)|[no-regex-spaces](https://github.com/buzinas/tslint-eslint-rules/blob/master/src/docs/rules/noRegexSpacesRule.md)|disallow multiple spaces in a regular expression literal (recommended)|
-|:white_check_mark:|[no-sparse-arrays](http://eslint.org/docs/rules/no-sparse-arrays)|[no-sparse-arrays](https://github.com/buzinas/tslint-eslint-rules/blob/master/src/docs/rules/noSparseArraysRule.md)|disallow sparse arrays (recommended)|
+|:white_check_mark:|[no-sparse-arrays](http://eslint.org/docs/rules/no-sparse-arrays)|[ter-no-sparse-arrays](https://github.com/buzinas/tslint-eslint-rules/blob/master/src/docs/rules/terNoSparseArraysRule.md)|disallow sparse arrays (recommended)|
 |:white_check_mark:|[no-unexpected-multiline](http://eslint.org/docs/rules/no-unexpected-multiline)|[no-unexpected-multiline](https://github.com/buzinas/tslint-eslint-rules/blob/master/src/docs/rules/noUnexpectedMultilineRule.md)|Avoid code that looks like two expressions but is actually one|
 |:no_entry_sign:|[no-unreachable](http://eslint.org/docs/rules/no-unreachable)|Not applicable|disallow unreachable statements after a return, throw, continue, or break statement (recommended)|
 |:ballot_box_with_check:|[no-unsafe-finally](http://eslint.org/docs/rules/no-unsafe-finally)|[no-unsafe-finally](https://palantir.github.io/tslint/rules/no-unsafe-finally)|disallow control flow statements in finally blocks (recommended)|
@@ -191,7 +191,7 @@ These rules have to do with variable declarations.
 |:no_entry_sign:|[no-undef](http://eslint.org/docs/rules/no-undef)|Not applicable|disallow use of undeclared variables unless mentioned in a `/*global */` block (recommended)|
 |:x:|[no-undef-init](http://eslint.org/docs/rules/no-undef-init)|no-undef-init|disallow use of undefined when initializing variables|
 |:x:|[no-undefined](http://eslint.org/docs/rules/no-undefined)|no-undefined|disallow use of `undefined` variable|
-|:no_entry_sign:|[no-unused-vars](http://eslint.org/docs/rules/no-unused-vars)|Not applicable|[**DEPRECATED**: [no-unused-variable](https://github.com/palantir/tslint/issues/1481)] disallow declaration of variables that are not used in the code (recommended).|
+|:ballot_box_with_check:|[no-unused-vars](http://eslint.org/docs/rules/no-unused-vars)|[no-unused-variable](https://palantir.github.io/tslint/rules/no-unused-variable/)|disallow unused variables (recommended).|
 |:ballot_box_with_check:|[no-use-before-define](http://eslint.org/docs/rules/no-use-before-define)|[no-use-before-declare](http://palantir.github.io/tslint/rules/no-use-before-declare)|disallow use of variables before they are defined|
 
 ### Node.js and CommonJS
@@ -226,6 +226,7 @@ These rules are purely matters of style and are quite subjective.
 |:x:|[computed-property-spacing](http://eslint.org/docs/rules/computed-property-spacing)|computed-property-spacing|require or disallow padding inside computed properties|
 |:x:|[consistent-this](http://eslint.org/docs/rules/consistent-this)|consistent-this|enforce consistent naming when capturing the current execution context|
 |:ballot_box_with_check:|[eol-last](http://eslint.org/docs/rules/eol-last)|[eofline](https://palantir.github.io/tslint/rules/eofline)|enforce newline at the end of file, with no multiple empty lines|
+|:white_check_mark:|[func-call-spacing](http://eslint.org/docs/rules/func-call-spacing)|[ter-func-call-spacing](https://github.com/buzinas/tslint-eslint-rules/blob/master/src/docs/rules/terFuncCallSpacingRule.md)|require or disallow spacing between function identifiers and their invocations|
 |:x:|[func-names](http://eslint.org/docs/rules/func-names)|func-names|require function expressions to have a name|
 |:x:|[func-style](http://eslint.org/docs/rules/func-style)|func-style|enforce use of function declarations or expressions|
 |:x:|[id-blacklist](http://eslint.org/docs/rules/id-blacklist)|id-blacklist|disallow certain identifiers to prevent them being used|

gulpfile.js

@@ -52,7 +52,7 @@ gulp.task('lint', function lint() {
   return gulp
     .src(SRC_FOLDER)
     .pipe(tslint({
-      formatter: 'verbose',
+      formatter: 'stylish',
     }))
     .pipe(tslint.report());
 });

index.js

@@ -1,5 +1,5 @@
 // unopinionated extensable tslint configuration
 // loads rules for extending packages, but does not enable any
 module.exports = {
-    rulesDirectory: "dist/rules",
+    rulesDirectory: "./dist/rules",
 };

package.json

@@ -1,6 +1,6 @@
 {
   "name": "tslint-eslint-rules",
-  "version": "3.5.1",
+  "version": "4.0.0",
   "description": "Improve your TSLint with the missing ESLint Rules",
   "main": "index.js",
   "scripts": {
@@ -25,22 +25,25 @@
   "license": "MIT",
   "devDependencies": {
     "@types/benchmark": "^1.0.30",
+    "@types/node": "^7.0.12",
     "benchmark": "^2.1.3",
     "chai": "^3.5.0",
     "es6-promise": "^4.0.4",
     "gulp": "^3.9.1",
     "gulp-sourcemaps": "^1.6.0",
     "gulp-spawn-mocha": "^2.2.2",
-    "gulp-tslint": "^7.0.0",
-    "gulp-typescript": "^3.0.1",
-    "tslint": "^4.0.0",
-    "typescript": "^2.0.3",
+    "gulp-tslint": "^8.0.0",
+    "gulp-typescript": "^3.1.6",
+    "tslint": "^5.0.0",
+    "typescript": "^2.2.0",
     "yargs": "^5.0.0"
   },
   "peerDependencies": {
-    "tslint": "^4.5.0"
+    "tslint": "^5.0.0"
   },
   "dependencies": {
-    "doctrine": "^0.7.2"
+    "doctrine": "^0.7.2",
+    "tslib": "^1.0.0",
+    "tsutils": "^1.4.0"
   }
 }

src/docs/rules/noIrregularWhitespaceRule.md

@@ -0,0 +1,61 @@
+<!-- Start:AutoDoc:: Modify `src/readme/rules.ts` and run `gulp readme` to update block -->
+## ter-func-call-spacing (ESLint: [func-call-spacing](http://eslint.org/docs/rules/func-call-spacing))
+[![rule_source](https://img.shields.io/badge/%F0%9F%93%8F%20rule-source-green.svg)](https://github.com/buzinas/tslint-eslint-rules/blob/master/src/rules/terFuncCallSpacingRule.ts)
+[![test_source](https://img.shields.io/badge/%F0%9F%93%98%20test-source-blue.svg)](https://github.com/buzinas/tslint-eslint-rules/blob/master/src/test/rules/terFuncCallSpacingRuleTests.ts)
+
+require or disallow spacing between function identifiers and their invocations
+
+#### Rationale
+
+This rule will enforce consistency of spacing in function calls,
+by disallowing or requiring one or more spaces before the open paren.
+
+### Config
+
+This rule has a string option:
+
+* `"never"` (default) disallows space between the function name and the opening parenthesis.
+* `"always"` requires space between the function name and the opening parenthesis.
+
+Further, in `"always"` mode, a second object option is available that contains a single boolean `allowNewlines` property.
+
+#### Examples
+
+```json
+"ter-func-call-spacing": [true]
+```
+
+```json
+"ter-func-call-spacing": [true, "always"]
+```
+
+```json
+"ter-func-call-spacing": [true, "always", { allowNewlines: true }]
+```
+#### Schema
+
+```json
+{
+  "type": "array",
+  "items": [
+    {
+      "enum": [
+        "always",
+        "never"
+      ]
+    },
+    {
+      "type": "object",
+      "properties": {
+        "allowNewlines": {
+          "type": "boolean"
+        }
+      },
+      "additionalProperties": false
+    }
+  ],
+  "minItems": 0,
+  "maxItems": 2
+}
+```
+<!-- End:AutoDoc -->

src/docs/rules/noSparseArraysRule.md

@@ -15,7 +15,7 @@ cleaner diffs in version control, and easier programmatic manipulation.
 The string 'tab' or an integer indicating the number of spaces to use per tab.
 
 An object may be provided to fine tune the indentation rules:
-      
+
   * `"SwitchCase"` (default: 0) enforces indentation level for `case` clauses in
                      `switch` statements
   * `"VariableDeclarator"` (default: 1) enforces indentation level for `var` declarators;
@@ -62,7 +62,7 @@ An object may be provided to fine tune the indentation rules:
       "body": 1
     }
   }
-]      
+]
 ```
 #### Schema