An in-range update of sass is breaking the build 🚨

[greenkeeper]

The dependency sass was updated from 1.22.7 to 1.22.8.

🚨 View failing branch.

This version is covered by your current version range and after updating it in your project the build failed.

sass is a direct dependency of this project, and it is very likely causing it to break. If other packages depend on yours, this update is probably also breaking those in turn.

Status Details

• ❌ continuous-integration/travis-ci/push: The Travis CI build failed (Details).

---

Release Notes for Dart Sass 1.22.8

To install Dart Sass 1.22.8, download one of the packages above and add it to your PATH, or see the Sass website for full installation instructions.

Changes

JavaScript API

• Don't crash when running in a directory whose name contains URL-sensitive characters.

• Drop support for Node Carbon (8.x), which doesn't support url.pathToFileURL.

See the full changelog for changes in earlier releases.

FAQ and help

There is a collection of frequently asked questions. If those don’t help, you can always ask the humans behind Greenkeeper.

---

Your Greenkeeper Bot 🌴

[greenkeeper]

After pinning to 1.22.7 your tests are passing again. Downgrade this dependency 📌.

[FND]

Drop support for Node Carbon (8.x), which doesn't support url.pathToFileURL.

https://github.com/sass/dart-sass/blob/master/CHANGELOG.md

Doesn't seem very semver; not sure what we should do about this.

[moonglum]

Maintenance for Node 8 runs until the end of the year. Why does nobody care about things like that? 😭 Typical minor release change...

I guess the only option here is to make Node 10 a requirement for faucet-pipeline-sass. This is the core library, it can't be replaced... 😒

[FND]

Yeah, I agree - realistically, I doubt any of our users are stuck with Node v8.

[moonglum]

Agreed. But we still need to make it a... breaking change, right? So release 2.0.0?

[FND]

Well, given that faucet-sass's package.json uses "sass": "~1.22.3", this is a problem for 1.x users either way, so in that regard, faucet-sass v2.0 would effectively be cosmetic?

[moonglum]

True 😭 Ok. So we release a new version 1.4.0 that adjusts the travis.yml and sets

"engines": {
  "node": ">=10"
},

Right?

[FND]

I suppose so, yeah.

[moonglum]

This version of sass has been unpublished. The authors were not aware that this is a breaking change (:suspect:) and reverted it. Here is the explanation

Going forward, we'll avoid dropping support for Node versions that we already support as long as they're still in the long-term service window.