tslint - npm Package Compare versions

Comparing version 0.4.4 to 0.4.5

build/formatters/jsonFormatter.js

@@ -0,0 +0,0 @@ var __extends = this.__extends || function (d, b) {

build/formatters/proseFormatter.js

@@ -0,0 +0,0 @@ var __extends = this.__extends || function (d, b) {

build/formatters/verboseFormatter.js

@@ -0,0 +0,0 @@ var __extends = this.__extends || function (d, b) {

build/rules/banRule.js

@@ -0,0 +0,0 @@ var __extends = this.__extends || function (d, b) {

build/rules/classNameRule.js

@@ -0,0 +0,0 @@ var __extends = this.__extends || function (d, b) {

build/rules/commentFormatRule.js

@@ -0,0 +0,0 @@ var __extends = this.__extends || function (d, b) {

build/rules/curlyRule.js

@@ -0,0 +0,0 @@ var __extends = this.__extends || function (d, b) {

build/rules/eoflineRule.js

@@ -0,0 +0,0 @@ var __extends = this.__extends || function (d, b) {

build/rules/forinRule.js

@@ -0,0 +0,0 @@ var __extends = this.__extends || function (d, b) {

build/rules/indentRule.js

@@ -0,0 +0,0 @@ var __extends = this.__extends || function (d, b) {

build/rules/interfaceNameRule.js

@@ -0,0 +0,0 @@ var __extends = this.__extends || function (d, b) {

build/rules/jsdocFormatRule.js

@@ -0,0 +0,0 @@ var __extends = this.__extends || function (d, b) {

build/rules/labelPositionRule.js

@@ -0,0 +0,0 @@ var __extends = this.__extends || function (d, b) {

build/rules/labelUndefinedRule.js

@@ -0,0 +0,0 @@ var __extends = this.__extends || function (d, b) {

build/rules/maxLineLengthRule.js

@@ -0,0 +0,0 @@ var __extends = this.__extends || function (d, b) {

build/rules/noArgRule.js

@@ -0,0 +0,0 @@ var __extends = this.__extends || function (d, b) {

build/rules/noBitwiseRule.js

@@ -0,0 +0,0 @@ var __extends = this.__extends || function (d, b) {

build/rules/noConsecutiveBlankLinesRule.js

@@ -0,0 +0,0 @@ var __extends = this.__extends || function (d, b) {

build/rules/noConsoleRule.js

@@ -0,0 +0,0 @@ var __extends = this.__extends || function (d, b) {

build/rules/noConstructRule.js

@@ -0,0 +0,0 @@ var __extends = this.__extends || function (d, b) {

build/rules/noDebuggerRule.js

@@ -0,0 +0,0 @@ var __extends = this.__extends || function (d, b) {

build/rules/noDuplicateKeyRule.js

@@ -0,0 +0,0 @@ var __extends = this.__extends || function (d, b) {

build/rules/noDuplicateVariableRule.js

@@ -0,0 +0,0 @@ var __extends = this.__extends || function (d, b) {

build/rules/noEmptyRule.js

@@ -0,0 +0,0 @@ var __extends = this.__extends || function (d, b) {

build/rules/noEvalRule.js

@@ -0,0 +0,0 @@ var __extends = this.__extends || function (d, b) {

build/rules/noStringLiteralRule.js

@@ -0,0 +0,0 @@ var __extends = this.__extends || function (d, b) {

build/rules/noTrailingCommaRule.js

@@ -0,0 +0,0 @@ var __extends = this.__extends || function (d, b) {

build/rules/noTrailingWhitespaceRule.js

@@ -0,0 +0,0 @@ var __extends = this.__extends || function (d, b) {

build/rules/noUnreachableRule.js

@@ -0,0 +0,0 @@ var __extends = this.__extends || function (d, b) {

build/rules/noUnusedVariableRule.js

@@ -7,2 +7,4 @@ var __extends = this.__extends || function (d, b) {
 };
+var OPTION_CHECK_PARAMETERS = "check-parameters";
+
 var Rule = (function (_super) {
@@ -83,5 +85,5 @@ __extends(Rule, _super);
         var variableName = node.identifier.text();
-        var position = this.positionAfter(node.modifiers);
+        var position = this.positionAfter(node.dotDotDotToken, node.modifiers) + node.leadingTriviaWidth();
 
-        if (!this.hasModifier(node.modifiers, "public") && !this.skipParameterDeclaration) {
+        if (!this.hasModifier(node.modifiers, "public") && !this.skipParameterDeclaration && this.hasOption(OPTION_CHECK_PARAMETERS)) {
             this.validateReferencesForVariable(variableName, position);
@@ -88,0 +90,0 @@ }

build/rules/noUseBeforeDeclareRule.js

@@ -0,0 +0,0 @@ var __extends = this.__extends || function (d, b) {

build/rules/oneLineRule.js

@@ -0,0 +0,0 @@ var __extends = this.__extends || function (d, b) {

build/rules/quotemarkRule.js

@@ -0,0 +0,0 @@ var __extends = this.__extends || function (d, b) {

build/rules/radixRule.js

@@ -0,0 +0,0 @@ var __extends = this.__extends || function (d, b) {

build/rules/semicolonRule.js

@@ -0,0 +0,0 @@ var __extends = this.__extends || function (d, b) {

build/rules/tripleEqualsRule.js

@@ -0,0 +0,0 @@ var __extends = this.__extends || function (d, b) {

build/rules/typedefRule.js

@@ -0,0 +0,0 @@ var __extends = this.__extends || function (d, b) {

build/rules/typedefWhitespaceRule.js

@@ -0,0 +0,0 @@ var __extends = this.__extends || function (d, b) {

build/rules/useStrictRule.js

@@ -0,0 +0,0 @@ var __extends = this.__extends || function (d, b) {

build/rules/variableNameRule.js

@@ -0,0 +0,0 @@ var __extends = this.__extends || function (d, b) {

build/rules/whitespaceRule.js

@@ -0,0 +0,0 @@ var __extends = this.__extends || function (d, b) {

CHANGELOG.md

 Change Log
 ===
 
+v0.4.5
+---
+* [feature] `no-unused-variable` no longer checks parameters by defualt. Parameters are now only checked if the `check-parameters` option is set.
+* [bug] `no-unused-variable` parameter check no longer fails on variable argument parameters (like ...args) and on cases where the parameters are broken up by newlines.
+
 v0.4.4
@@ -5,0 +10,0 @@ ---

package.json

 {
   "name": "tslint",
-  "version": "0.4.4",
+  "version": "0.4.5",
   "description": "a static analysis linter for TypeScript",
@@ -5,0 +5,0 @@ "bin": {

README.md

@@ -41,2 +41,4 @@ TSLint [![NPM version](https://badge.fury.io/js/tslint.png)](http://badge.fury.io/js/tslint) [![Builds](https://api.travis-ci.org/repositories/palantir/tslint.png?branch=master)](https://travis-ci.org/palantir/tslint)
 * `no-unused-variable` disallows unused imports, variables, functions and private class members.
+    * `"check-parameters"` disallows unused function and constructor parameters.
+        * NOTE: this option is experimental and does not work with classes that use abstract method declarations, among other things. Use at your own risk.
 * `no-unreachable` disallows unreachable code after `break`, `catch`, `throw`, and `return` statements.
@@ -43,0 +45,0 @@ * `no-use-before-declare` disallows usage of variables before their declaration.

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

Lines of code

125458

Number of lines in readme file

197

increased by1.03%

Worsened metrics

Total package byte prevSize

7036152

decreased by-2.08%

No dependency changes