Skip to content

Conversation

@isaacs
Copy link
Contributor

@isaacs isaacs commented Feb 26, 2020

Fix #60 (for the legacy branch)
Fix #61

@isaacs isaacs requested a review from darcyclarke February 26, 2020 22:22
Fix #61 This should not be ported to the latest branch, as Node.js v6 support was dropped there anyway.
@billneff79
Copy link

@isaacs - I put forward an alternative fix in PR #64 that gets rid of url.URL all together. I don't care which gets merged, as long as one of the fixes does

isaacs added a commit that referenced this pull request Feb 26, 2020
Fix #61 This should not be ported to the latest branch, as Node.js v6 support was dropped there anyway. PR-URL: #62 Credit: @isaacs Close: #62 Reviewed-by: @isaacs
@darcyclarke
Copy link
Contributor

Closing as both PRs w/ this work have landed against latest & legacy

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants