al-the-x / cloudinary-sass Goto Github PK

Hello! Thanks for making such a useful library, it's exactly what I'm looking for 😄 I'm hoping to use it but when I import the package, the compiler throws this error:

SassError: This module and the new module both define a variable named "$RESPONSIVE-IMAGE-SET-PREFIXED".
3 │ @use "./responsive" as *;

For context I'm using these 2 sass libs: sass 1.49.9 (dart sass) and sass-loader 12.4.0.

The problem seems to be the 2 "use" statements that aren't namespaced in _cloudinary.scss:

// FIXME: Don't pollute the main namespace once `libsass  / `sass-loader` recognize `@use`
@use "./string" as *;
@use "./responsive" as *;

Since LibSass and Node Sass are now deprecated, what do you think about updating these lines to be namespaced? I tried it out and it seemed to fix the problem, I'm happy to submit a PR to that effect.

Let me know, and thanks again for sharing this work!