Closed pepkin88 closed 6 years ago
This looks good to me, but I'd like a second person to test this on Windows (I assume you did?) just to be sure (and to ensure there are no lurking second bugs hiding behind the one you fixed).
Works on my windows
Yes, I checked both os.homedir!
and process.env.USERPROFILE
. Windows is my main platform, although I do tests and builds on a virtual Ubuntu machine.
Fix for #897
os.homedir
was added in Node v2.3.0, fallback values ofprocess.env.HOME
andprocess.env.USERPROFILE
works for POSIX and Windows