-
Notifications
You must be signed in to change notification settings - Fork 20
Please remove dependency on os-homedir #11
Comments
As of this writing |
@iarna, fair enough you want to support older nodejs versions, but are you saying its the responaibility of random bug reporters to fix critical bugs in your own code? Or are you saying the bug in os-homedir somehow does not affect your code? |
As for my usecase, debian has on older osenv without os-homedir dependency and since we already have a newer nodejs, we don't need os-homedir. So I don't need to fix os-homedir. It is upto you, how you want to keep your code in good shape. |
IMHO, this case should be taken of specially. We can't rely on an underlying implementation known to be defected. Can we implement our own polyfill on the npm side? |
I don't see this as critical as you seem to (obviously seeing as I'm just getting back to this now). We have a very, very small team and so our work is all about triage. I wasn't insisting that you do any work, just telling you what we could and could not accept. @welladamm We could patch |
Its has serious unfixed issues https://github.com/sindresorhus/os-homedir/issues/4 expand-tilde already removed this dependency.
jonschlinkert/expand-tilde@e13f94a
The text was updated successfully, but these errors were encountered: