-
Notifications
You must be signed in to change notification settings - Fork 303
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
Update dependencies #447
Comments
Thanks, @adam-lynch, for spending some time on this project again. 👍 Since you're intending to upgrade the dependencies, could you please take a look at the If you run
|
Will do thanks
…On Sat, 20 May 2017, 19:37 Sebastian Meyer, ***@***.***> wrote:
Thanks, @adam-lynch <https://github.com/adam-lynch>, for spending some
time on this project again. 👍
Since you're intending to upgrade the dependencies, could you please take
a look at the temp dependency? This one is causing an annoying
deprecation message on newer node versions.
If you run node --trace-deprecation ..., you can see that it's coming
from there.
This has already been fixed in a newer version two years ago, but the
dependency doesn't seem to be maintained anymore, so maybe you want to take
a look at another one, if it's necessary.
(node:12673) DeprecationWarning: os.tmpDir() is deprecated. Use os.tmpdir() instead.
at Object.<anonymous> (/home/basti/foo/node_modules/temp/lib/temp.js:170:45)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#447 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ABXHKW0iThQSoHb2YIgBwhYPiXsl-fByks5r7zL7gaJpZM4NhXee>
.
|
@bastimeyer like what? I don't see it when using Node 6.10.3 |
The one I already posted:
This has been deprecated in node 7: |
I meant which version. Thanks. |
Add greenkeeper preferably. Edit: it's already hooked up but broken possibly. Need to more a few PRs too... depends on #446.
The text was updated successfully, but these errors were encountered: