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

Update node-sass for compatibility with Node.js 10 #9404

Merged
merged 1 commit into from
May 16, 2018

Conversation

teohhanhui
Copy link
Contributor

@teohhanhui teohhanhui commented May 14, 2018

Q A
Branch? 1.0
Bug fix? yes?
New feature? no
BC breaks? no
Deprecations? no
Related tickets N/A
License MIT

Node.js 10 compatibility was introduced in node-sass v4.9.0

@teohhanhui
Copy link
Contributor Author

teohhanhui commented May 14, 2018

Note: gulp-sass v4.0.0 dropped support for old Node.js versions. If that's not fine for us, we could perhaps stick to gulp-sass v3.2.1 for now.

But it's probably not worth it to maintain compatibility with unsupported old Node versions: https://github.com/nodejs/Release#release-schedule

@pamil pamil added the Maintenance CI configurations, READMEs, releases, etc. label May 16, 2018
@pamil
Copy link
Contributor

pamil commented May 16, 2018

I'd say we have the same policy as for PHP - supporting only officially supported versions, so I'm fine with that.

@pamil pamil merged commit 2e36854 into Sylius:1.0 May 16, 2018
@pamil
Copy link
Contributor

pamil commented May 16, 2018

Thank you, Teoh! :)

@teohhanhui teohhanhui deleted the update/node-sass branch May 16, 2018 09:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Maintenance CI configurations, READMEs, releases, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants