-
Notifications
You must be signed in to change notification settings - Fork 317
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
ncu broken with Node 8.0.0-rc1 #355
Comments
This appears to be an issue with the |
Also appears on my laptop. And now node 8 is not RC, it is officially released. |
same here |
yup, still broken in Node.js 8 stable as well.. |
exactly |
Same here as well! |
Getting this issue today after a Node update |
same on node8@osx |
same problem after installing node 8.0 |
I copied the fix from the PR to require-dir https://github.com/aseemk/requireDir/pull/46/files into my global install of npm-check-updates, and it fixed the issue. Guess we'll have to wait until the PR is merged and there's a newer version of require-dir. |
same =( |
+1 |
Take a look at @jharris4 's pull file, simply replace the line and things will be working as intended until it's merged. |
Glad it helped! Although it wasn't my PR, I just found it. :-) |
Thanks, im using the require-dir PR patch now |
requre-dir |
I might have recommended bumping the semver of require-dir in package.json to |
@jharris4 |
@raineorshine right, but my point was that |
Confirmed: I needed to uninstall and then re-install. npm uninstall npm-check-updates -g && npm install npm-check-updates -g |
Ah, you are right! I think patches in general can be assumed to fix bugs within the same major/minor version. If you lock your dependencies and do not allow patch upgrades, you miss out on bug fixes. |
@mathiasbynens Just FYI, with npm 5.0.0 (which I just upgraded to) there are some problems uninstalling global packages. I had to re-install without uninstalling, but it worked. Here's a link to the issue on npm: npm/npm#16738 |
|
@webjay Are you using npm 5.0.0 with npm-check-updates previously installed with npm 4.x? |
@jharris4 I'm using npm 4.2.0 and node v8 on a Mac. |
@webjay I guessed as much. I'm using homebrew to install node on a Mac too :-) |
@webjay Thank you! It works. |
fixes here: #362 |
I'm on OS X with Node installed by brew and npm installed separately. The uninstall also didn't work but |
El Capitan 10.11.6 / homebrew upgrade node v8.1.2 / npm v5.0.3 the attempt for `Unhandled rejection Error: Argument #2: Expected string but got nullf8dfac7a8f7b npm ERR! cb() never called! npm ERR! This is an error with npm itself. Please report this error at: npm ERR! A complete log of this run can be found in: After removing the project |
sudo apt-get remove nodejs sudo apt-get remove npm Then go to /etc/apt/sources.list.d and remove any node list if you have. Then do a sudo apt-get update Check for any .npm or .node folder in your home folder and delete those. If you type which node you can see the location of the node. Try which nodejs and which npm too Now install node.js curl -sL https://deb.nodesource.com/setup_6.x | sudo -E bash - |
I was able to resolve this quick because I have NVM installed. I just switched to a different version of Node. I was using |
After replacing Node 7.10 with 8.0.0-rc.1 I get the following stacktrace when invoking
ncu
:The text was updated successfully, but these errors were encountered: