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

In win32 platform, %APPDATA%\npm\etc\ should coresspond to /etc. #88

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

chi-bd
Copy link

@chi-bd chi-bd commented Nov 20, 2016

Global settings of npm are not referenced in win32 platform.
'npm config --global' refers to %APPDIR%\npm\etc\npmrc, so %APPDATA%\npm\etc\ should coresspond to /etc.

('npm config --global' refers to %APPDIR%\npm\etc\npmrc)
@dominictarr
Copy link
Owner

are you using this to load npm's configuration?

@chi-bd
Copy link
Author

chi-bd commented Nov 20, 2016

I've not used directly, but the "get-proxy" is used.

There is a report that an error occurs using "vue-cli" via proxy(npm config set proxy), so I surveyed and found this problem at rc.
(vue-cli -> download-git-repo -> caw -> get-proxy -> rc)

@dominictarr
Copy link
Owner

oh, this should be fixed in get-proxy
kevva/get-proxy#5

I can't change this behavior because I do not know if it might break something else.
rc is not intended to support npm's particular config behavior. use the npm-config module if you want to read npm's config

@kevva
Copy link

kevva commented Jul 21, 2017

This should be fixed in kevva/get-proxy@2869a17.

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.

None yet

3 participants