search

yuschick / stylelint-plugin-defensive-css

A Stylelint plugin for enforcing defensive CSS best practices.
https://defensivecss.dev/
MIT License
139 stars 4 forks source link

Include default value for `flex-wrap` #20

Closed crstauf closed 1 year ago

crstauf commented 1 year ago

Is there a way I can adjust the error message (per project) so that the default value for flex-wrap is included? I expect that junior devs on my team will see the error, and add flex-wrap: wrap without thinking, just to resolve the error.

If not, I may have to run without the rule in place, to prevent other problems.

yuschick commented 1 year ago

There's no support at this time for custom messaging. But I am happy to take suggestions on the message that would be more clear. I agree, it could mention the default nowrap and needing to explicitly define that if that's what is intended.

yuschick commented 1 year ago

I did a pass on all the error messages in #23. You can see the specific flex-wrapping error update here.