grunt-ts - npm Package Compare versions

Comparing version 1.5.0 to 1.5.1

package.json

@@ -5,3 +5,3 @@ {
     "description": "Compile and manage your TypeScript project",    
-    "version": "1.5.0",
+    "version": "1.5.1",
     "homepage": "https://github.com/basarat/grunt-ts",
@@ -8,0 +8,0 @@ "repository": {

README.md

@@ -124,2 +124,3 @@ grunt-ts
 This is useful in some cases and the most common way is using AMD
+* You want the ability to individually compile only changed files (for a faster dev-compile-run cycle)
 * However, File order doesn't matter to you, even when you have inter file depenendency (e.g. AngularJS runtime Dependency injection)
@@ -213,3 +214,3 @@
                     declaration: false,            // true | false (default)
-                    comments: false                // true | false (default)
+                    removeComments: true           // true (default) | false
                 },
@@ -216,0 +217,0 @@ },

tasks/ts.js

@@ -553,3 +553,5 @@ /// <reference path="../defs/node/node.d.ts"/>
 
-        if (options.removeComments !== null && options.comments !== null) {
+        if (options.removeComments === null) {
+            options.removeComments = !options.comments;
+        } else if (options.comments !== null) {
             console.warn('WARNING: Option "comments" and "removeComments" should not be used together'.magenta);
@@ -562,6 +564,4 @@ if (options.removeComments === options.comments) {
         }
+        options.removeComments = !!options.removeComments;
 
-        // Remove comments based on the removeComments flag first then based on the comments flag, otherwise true
-        options.removeComments = options.removeComments === true || options.comments === false || true;
-
         // Was the whole process successful
@@ -568,0 +568,0 @@ var success = true;

tasks/ts.ts

@@ -635,3 +635,5 @@ /// <reference path="../defs/node/node.d.ts"/>
         // Remove comments based on the removeComments flag first then based on the comments flag, otherwise true
-        if (options.removeComments !== null && options.comments !== null) {
+        if (options.removeComments === null) {
+            options.removeComments = !options.comments;
+        } else if (options.comments !== null) {
             console.warn('WARNING: Option "comments" and "removeComments" should not be used together'.magenta);
@@ -646,4 +648,3 @@ if (options.removeComments === options.comments) {
         }
-        // Remove comments based on the removeComments flag first then based on the comments flag, otherwise true
-        options.removeComments = options.removeComments === true || options.comments === false || true;
+        options.removeComments = !!options.removeComments;
 
@@ -650,0 +651,0 @@ // Was the whole process successful

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

Number of package files

21

increased by10.53%

Lines of code

6672

increased by0.04%

Number of lines in readme file

257

increased by0.39%

Worsened metrics

Total package byte prevSize

304411

decreased by-1.09%

No dependency changes