sass / migrator

Tool for migrating stylesheets to new Sass versions
MIT License
89 stars 11 forks source link

Built-in glob resolution broken on Node #189

Closed frattaro closed 3 years ago

frattaro commented 3 years ago

Running node 14.x on windows 10

Ran: npm i -g sass-migrator sass-migrator division **/*

Error> Unsupported operation: Recursive list is not supported in Node.js.

This is following the steps at https://sass-lang.com/documentation/breaking-changes/slash-div#automatic-migration

chouette-cc commented 3 years ago

same issue

jathak commented 3 years ago

It looks like's the migrator's support for globs is broken when running on Node. This isn't usually an issue on Mac and Linux, since shells will resolve the globs themselves, but Windows expects programs to do it themselves.

I'll try to find a fix, but for the time being, you can download the standalone release of the migrator here and use that, which should work fine.

You'll also want to call it with sass-migrator division **/*.scss instead of what's listed in the documentation (I have a PR to update this out for review now in https://github.com/sass/sass-site/pull/545).

frattaro commented 3 years ago

Thanks -- tried that immediately after, filed #190 :)