Skip to content
This repository was archived by the owner on Feb 18, 2022. It is now read-only.

warnings used without a fallback #56

Closed
Memoyr opened this issue Nov 7, 2016 · 11 comments
Closed

warnings used without a fallback #56

Memoyr opened this issue Nov 7, 2016 · 11 comments

Comments

@Memoyr
Copy link

Memoyr commented Nov 7, 2016

is it possible to deactivate this warning, "used without a fallback " ?
thanks

@Skotu
Copy link

Skotu commented Feb 6, 2017

Yes, it's annoying me a lot, so would like to have some option to deactivate it.

@peterschmiz
Copy link

As far as I could understand - based on the source code - neither strict or warnings options key has any effect on the warning :(

@archansel
Copy link

+1

@bradennapier
Copy link

+1000

I have about 180 copies of 40 variables because I need to include :root() in EVERY single css file due to this problem.

I have a root level of variables and 1000's of css files in our app that all expect to reference the variables (theme) in the globally included css file.

This completely kills the app since I either geet thousands of lines of warnings on every save or I get thousands of lines of redundant css

image

@s-panferov
Copy link

Please add an ability to disable the warning. In out product we use both "real" and "fake" css-properties, and this warning is annoying.

@LeaVerou
Copy link

Add me to the list of people wanting to suppress this warning. I've used warnings: false, doesn't do anything.

@LeaVerou
Copy link

If anyone wants to work on a PR, the relevant line seems to be:

"variable '" + name + "' is undefined and used without a fallback",

However, warnings is not defined in that scope, so it doesn't seem as trivial as just wrapping that with an if (warnings) { ... } :/

@jonathantneal
Copy link
Member

Would #67 also work as a solution, @LeaVerou? I’ve requested permission to resolve and merge it ASAP.

@LeaVerou
Copy link

@jonathantneal the code is a bit awkward, but it should work.

@jonathantneal
Copy link
Member

Resolved via #67, published as 6.1.0.

@SuperOl3g
Copy link
Contributor

SuperOl3g commented Jul 10, 2017

And how about to add the ability to turn this warning into error?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

9 participants