@fimbul/wotan - npm Package Compare versions

Comparing version 0.4.0-dev.20180301 to 0.4.0-dev.20180302

package.json

 {
   "name": "@fimbul/wotan",
-  "version": "0.4.0-dev.20180301",
+  "version": "0.4.0-dev.20180302",
   "description": "Pluggable TypeScript and JavaScript linter",
@@ -5,0 +5,0 @@ "bin": "bin/main.js",

README.md

@@ -46,4 +46,4 @@ # wotan
 `await-async-result` | Warns about not using the result of a call to an async function inside async functions. *requires type information* | TSLint's `no-floating-promises` requires you to specify a list of Promise names, it checks outside of async functions and only requires you to register the `onrejected` callback.
-`await-promise` | Finds uses of `await` on non-Promise values. Also checks `for await` loops. *requires type information* | Works for all `PromiseLike` and `Thenable` types out of the box without any configuration.
-`generator-yield` | Require at least one `yield` inside generator functions. | There's no similar TSLint rule.
+`await-only-promise` | Finds uses of `await` on non-Promise values. Also checks `for await` loops. *requires type information* | Works for all `PromiseLike` and `Thenable` types out of the box without any configuration.
+`generator-require-yield` | Require at least one `yield` inside generator functions. | There's no similar TSLint rule.
 `no-debugger` | Ban `debugger;` statements from your production code. | Performance!
@@ -87,3 +87,3 @@ `no-fallthrough` | Prevents unintentional fallthough in `switch` statements from one case to another. If the fallthrough is intended, add a comment that matches `/^\s*falls? ?through\b/i`. | Allows more comment variants such as `fallthrough` or `fall through`.
         "no-useless-assertion": "off",
-        "await-promise": "warn"
+        "await-only-promise": "warn"
     }
@@ -105,3 +105,3 @@ }
       no-useless-assertion: off
-      await-promise: warn
+      await-only-promise: warn
   - files: "*.spec.ts" # override the following rules for all *.spec.ts files in all directories
@@ -108,0 +108,0 @@ rules:

src/rules/prefer-object-spread.d.ts

@@ -6,3 +6,3 @@ import { TypedRule } from '../types';
     apply(): void;
-    private isSpreadableObject(node, i);
+    private isSpreadableObject(node);
 }

src/rules/prefer-object-spread.js

@@ -33,7 +33,10 @@ "use strict";
     }
-    isSpreadableObject(node, i) {
-        if (i === 0)
-            return true;
-        if (node.kind === ts.SyntaxKind.ThisKeyword || node.kind === ts.SyntaxKind.SpreadElement)
-            return false;
+    isSpreadableObject(node) {
+        switch (node.kind) {
+            case ts.SyntaxKind.ThisKeyword:
+            case ts.SyntaxKind.SpreadElement:
+                return false;
+            case ts.SyntaxKind.ObjectLiteralExpression:
+                return true;
+        }
         const type = this.checker.getTypeAtLocation(node);
@@ -40,0 +43,0 @@ if (type.flags & ts.TypeFlags.Any)

No alert changes

Improved metrics

Total package byte prevSize

407896

increased by0.04%

Lines of code

5185

increased by0.06%

No dependency changes