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

Use latest node-gyp #2819

Closed
wants to merge 3 commits into from
Closed

Use latest node-gyp #2819

wants to merge 3 commits into from

Conversation

jeremad
Copy link

@jeremad jeremad commented Jan 14, 2020

This gets rid of "python2-only" issues.

We also remove Node < 6 as those versions are not supported anymore. (even node 6 is not maintained)

ZoranPandovski and others added 2 commits November 17, 2019 17:07
Fix Unexpected token ) on the render example
This gets rid of "python2-only" issues
@jeremad jeremad requested a review from nschonni January 14, 2020 17:28
Node 4 is not supported any more.
@jeremad
Copy link
Author

jeremad commented Jan 22, 2020

I "fixed" the CI by removing older, unmaintained, node versions. Any chance to get a review on this?

@xzyfer
Copy link
Contributor

xzyfer commented Jan 22, 2020

This cannot land of master to the breaking changes. Could you set to PR to be against the v5 branch were it is likely to land.

@jeremad
Copy link
Author

jeremad commented Jan 22, 2020

Thank you!

@jeremad jeremad changed the base branch from master to v5 January 22, 2020 12:56
@jeremad
Copy link
Author

jeremad commented Jan 22, 2020

You actually already did it on the v5 branch. Thank you again.

@jeremad jeremad closed this Jan 22, 2020
@xzyfer
Copy link
Contributor

xzyfer commented Jan 22, 2020

Please rebase out all the commits except the package.json changes. You have conflicts now because we've made changes to supported node versions on the v5 branch.

@xzyfer
Copy link
Contributor

xzyfer commented Jan 22, 2020

Great. Thanks for the contribution. We hope to release v5 soon.

@jeremad
Copy link
Author

jeremad commented Mar 30, 2020

Any idea when the v5 will be released?

jiongle1 pushed a commit to scantist-ossops-m2/node-sass that referenced this pull request Apr 7, 2024
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.

3 participants