-
-
Notifications
You must be signed in to change notification settings - Fork 9
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
1.3.1 Failed on lower Node.js #9
Comments
same problem |
#10 same |
same |
1 similar comment
same |
same here... :( |
Same here, please revert - cannot build :( |
@isaacs I have to use Node 6.9.5 as have hardware & cannot update. |
We have the same issue on node version 6.10.0. |
For those still looking for solution, please follow the other commits referenced above. There's a better approach apparently, by specifying |
Hey, @himanshusinghs, just tried adding "minizlib": "1.2.2" to out package.json, but same result on Gitlab :( Tried generating new yarn.lock file too as we use yarn install on Gitlab |
@molinto check _requiredBy entries in package.json of minizlib to see what requires it. Tar module seems to allow version 1.2.2 and adding it to main file as exact version worked for me. |
@Srokap Cannot see any required in package.json (https://github.com/isaacs/minizlib/blob/master/package.json) or v1.2.2: Where am I looking please? |
@Srokap you probably wanted to mention |
Fixed on 1.3.2. |
@himanshusinghs I am on node 6 with bundled npm, so no package locks for me. I can check it by running:
I found that tar module was requiring |
because of:
I think this is a breaking change and should increase a major version.
The text was updated successfully, but these errors were encountered: