stylelint-processor-styled-components icon indicating copy to clipboard operation
stylelint-processor-styled-components copied to clipboard

Do I need this?

Open jfairley opened this issue 4 years ago • 2 comments

👋

Hi, I merely have a question here. I've been blindly using stylelint-processor-styled-components along with stylelint-config-styled-components because the documentation says so.

I was recently setting up a new project and noticed that – as far as I can tell – linting still works correct without stylelint-processor-styled-components.

I went on a quest to figure out what a stylelint-processor even is and I found this

Their use is discouraged favor of PostCSS syntaxes.

By the fact that it works, am I right to assume that stylelint-config-styled-components has been updated to no longer need stylelint-processor-styled-components? Or am I missing something?

Either way, it'd be really nice to update the README to explain when, if ever, adding stylelint-processor-styled-components is necessary.

jfairley avatar Nov 17 '20 23:11 jfairley

See https://github.com/styled-components/stylelint-processor-styled-components/issues/278 and https://github.com/stylelint/postcss-css-in-js/issues/3.

In summary, the future is stylelint PostCSS syntaxes. If you are new users and find stylelint works well for you, this processor is unnecessary.

chinesedfan avatar Nov 22 '20 13:11 chinesedfan

Thanks for the link @chinesedfan. I personally feel fine about removing stylelint-processor-styled-components from my CRA app.

IDK if you want this closed or left open for others, so I'll leave it open and let the maintainers decide.

jfairley avatar Nov 23 '20 00:11 jfairley