decomment - npm Package Compare versions

Comparing version 0.9.0 to 0.9.1

package.json

 {
   "name": "decomment",
-  "version": "0.9.0",
+  "version": "0.9.1",
   "description": "Removes comments from JSON/JavaScript, CSS/HTML, CPP/H, etc.",
@@ -45,10 +45,10 @@ "main": "lib/index.js",
   "dependencies": {
-    "esprima": "^4.0.0"
+    "esprima": "~4.0.0"
   },
   "devDependencies": {
-    "coveralls": "2.11",
-    "eslint": "^4.0.0",
-    "istanbul": "0.4",
-    "jasmine-node": "1.x"
+    "coveralls": "~2.11.16",
+    "eslint": "~4.7.0",
+    "istanbul": "~0.4.5",
+    "jasmine-node": "~1.14.5"
   }
 }

README.md

@@ -75,4 +75,4 @@ decomment
 * `true` - keep special multi-line comments that begin with:
- - `<!--[if` - for conditional comments in HTML
- - `/*!` - for everything else (other than HTML)
+  - `<!--[if` - for conditional comments in HTML
+  - `/*!` - for everything else (other than HTML)
 
@@ -79,0 +79,0 @@ Example:

test/multiSpec.js

@@ -193,2 +193,3 @@ 'use strict';
             expect(decomment('/*start' + LB + 'middle' + LB + 'end*/text')).toBe('text');
+            // TODO: This one seems invalid, should expect 'prefix- suffix', to avoid code fusion
             expect(decomment('prefix-/*start' + LB + 'middle' + LB + 'end*/suffix')).toBe('prefix-suffix');
@@ -209,2 +210,16 @@ });
 
+    // TODO: Code needs to be amended for these tests to pass:
+    /*
+    describe('when code is joined by comment on the same line', () => {
+        it('must not fuse the code', () => {
+            expect(decomment('var/*comment*\/name;')).toBe('var name;');
+        });
+    });
+
+    describe('when code is joined by comment across lines', () => {
+        it('must not fuse the code', () => {
+            expect(decomment('var/*multi' + LB + 'line*\/name;')).toBe('var name;');
+        });
+    });
+    */
 });

No alert changes

Improved metrics

Total package byte prevSize

58131

increased by1.1%

Lines of code

1117

increased by1.27%

Dependency changes

• Updatedesprima@~4.0.0