Skip to content
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

os.tmpDir() - deprecated #26 #27

Merged
merged 1 commit into from
Jan 17, 2017
Merged

os.tmpDir() - deprecated #26 #27

merged 1 commit into from
Jan 17, 2017

Conversation

rudidude86
Copy link

Full disclosure: Out of an abundance of caution, I just manually hot-dogged in the updated npm shrinkwrap entry for cache-swap because I didn't want to inadvertently break anything else with other dependencies. Not sure if that's frowned upon or not... is the preferred approach to just update all the things and hope nothing breaks?

* call [`os.tmpdir()`](https://nodejs.org/api/os.html#os_os_tmpdir) instead of the [deprecated `os.tmpDir()` alias](nodejs/node#6739)
* include newer version of cache-swap with [tmpDir -> tmpdir issue fixed](jgable/cache-swap@69fab2b)
@jgable jgable merged commit c00665d into jgable:master Jan 17, 2017
@jgable
Copy link
Owner

jgable commented Jan 17, 2017

Thanks for the PR, published as 0.1.0.

@jgable
Copy link
Owner

jgable commented Jan 17, 2017

And yes, the cache-swap update seems reasonable to include in this PR. Thanks again for taking the time to put the PR up.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants