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

Fix IPv6 issue on npm install #35

merged 4 commits into from Aug 3, 2018

Fix IPv6 issue on npm install #35

merged 4 commits into from Aug 3, 2018


Copy link

@valentin2105 valentin2105 commented Jul 30, 2018

Remove hardcoded IPv4 adress familly

@valentin2105 valentin2105 requested a review from as a code owner Jul 30, 2018
@valentin2105 valentin2105 changed the title Fix IPv6 issue ,… Fix IPv6 issue on npm install Jul 30, 2018
Valentin Ouvrard and others added 2 commits Jul 31, 2018
zkat approved these changes Aug 3, 2018
Copy link

@zkat zkat left a comment

👍 thanks for filing this!

@zkat zkat changed the base branch from latest to release-next Aug 3, 2018
@zkat zkat merged commit 84bfd23 into npm:release-next Aug 3, 2018
2 checks passed
2 checks passed
Travis CI - Pull Request Build Passed
continuous-integration/travis-ci/pr The Travis CI build passed
Copy link
Contributor Author

@valentin2105 valentin2105 commented Aug 6, 2018

Thanks for cleaning the stuffs 👍

@zkat zkat mentioned this pull request Aug 15, 2018
4 tasks
@zkat zkat mentioned this pull request Aug 29, 2018
4 tasks
isaacs added a commit that referenced this issue Aug 12, 2019
- [`9c93ac3`](npm/cmd-shim@9c93ac3)
  Handle environment variables properly

- [`2d277f8`](npm/cmd-shim@2d277f8)
  [#35](npm/cmd-shim#35) Fix 'no shebang' case
  by always providing `$basedir` in shell script

- [`adaf20b`](npm/cmd-shim@adaf20b)
  [#26](npm/cmd-shim#26) Fix `$*` causing an
  error when arguments contain parentheses

- [`49f0c13`](npm/cmd-shim@49f0c13)
  [#30](npm/cmd-shim#30) Fix paths for
  MSYS/MINGW bash ([@dscho](

- [`51a8af3`](npm/cmd-shim@51a8af3)
  [#34](npm/cmd-shim#34) Add proper support for
  PowerShell ([@ExE-Boss](
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants