NO_PROXY variable for npm

nodeshift-archived / ubi8-s2i-web-app

Apache License 2.0

26 stars 28 forks source link

Closed crmarques closed 5 years ago

crmarques commented 5 years ago

Patch to setup npm to use $no_proxy

helio-frota commented 5 years ago

LGTM. Thanks for the contribution. Also don't need to bother with the CI error for now, that unstable behavior happened before. https://travis-ci.org/nodeshift/centos7-s2i-web-app/builds/554295648#L388

lholmquist commented 5 years ago

Lets not forget to also bring this into the 12.x and 10.x branches