Skip to content
This repository has been archived by the owner on Jul 24, 2024. It is now read-only.

chore: update request package #2170

Closed
wants to merge 1 commit into from

Conversation

alexander-akait
Copy link

@xzyfer
Copy link
Contributor

xzyfer commented Nov 29, 2017 via email

@xzyfer xzyfer closed this Nov 29, 2017
@alexander-akait alexander-akait deleted the update-request branch November 29, 2017 12:30
xzyfer referenced this pull request Dec 21, 2017
request@2.81.0 breaks Node < 4 compat.
@rogeruiz rogeruiz mentioned this pull request Feb 20, 2018
3 tasks
@jc1arke
Copy link

jc1arke commented Feb 21, 2018

Just checking: since this is now being halted on downstream projects by NSP, when will version 5 be released?

Support for Node 4 as an LTS ends in basically 2 months (https://github.com/nodejs/Release#release-schedule), meaning that blocking the dependency update to cater for a soon-to-be-deprecated version just doesn't seem valid.

The lowest active LTS is 6.x, that should be the minimum not true?

martin-corevski added a commit to martin-corevski/webpack-basics that referenced this pull request Apr 27, 2018
Almost fixed hoek package, dependency of hawk -> request -> node-sass
request/request#2874
Waiting for version 5 of node-sass
sass/node-sass#2170
NBakaev added a commit to NBakaev/angular-essential-select that referenced this pull request May 5, 2018
@tflori
Copy link

tflori commented May 17, 2018

just my two cents: node-sass 4.9 requires node-gyp 3.3 and node-gyp 3.3 that also requires hawk 4.2.1 with the result that node-sass 4.9 already requires node >= 4 - so I don't understand the point in blocking an update because it is breaking node 0.10 support 😕

@nschonni
Copy link
Contributor

node-gyp is only used if the binary download fails, which is done through request

@sass sass locked as resolved and limited conversation to collaborators May 17, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants