common-shakeify - npm Package Compare versions

Comparing version 1.1.0 to 1.1.1

CHANGELOG.md

@@ -7,2 +7,5 @@ # common-shakeify change log
 
+## 1.1.1
+* Fixes infinite loop in new side-effect feature. ([#40](https://github.com/browserify/common-shakeify/pull/40))
+
 ## 1.1.0
@@ -9,0 +12,0 @@ * Entirely remove side-effect-free unused modules when they declare `sideEffects: false` in `package.json`. ([#31](https://github.com/browserify/common-shakeify/pull/31))

index.js

@@ -64,4 +64,6 @@ 'use strict'
         let dir = file
-        while (!pkg && (dir = path.dirname(dir))) {
+        let prevDir = null
+        while (!pkg && (dir = path.dirname(dir)) && prevDir !== dir) {
           pkg = packages.get(dir)
+          prevDir = dir
         }
@@ -68,0 +70,0 @@ return pkg && pkg.sideEffects === false ? false : true

package.json

 {
   "name": "common-shakeify",
-  "version": "1.1.0",
+  "version": "1.1.1",
   "description": "browserify tree shaking plugin using @indutny common-shake",
@@ -5,0 +5,0 @@ "main": "index.js",

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

Total package byte prevSize

46539

increased by0.42%

Lines of code

885

increased by0.23%

No dependency changes