Skip to content
This repository has been archived by the owner on May 14, 2021. It is now read-only.

Do I need this? #300

Open
jfairley opened this issue Nov 17, 2020 · 2 comments
Open

Do I need this? #300

jfairley opened this issue Nov 17, 2020 · 2 comments
Labels

Comments

@jfairley
Copy link

jfairley commented Nov 17, 2020

👋

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.

@chinesedfan
Copy link
Member

See #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.

@jfairley
Copy link
Author

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.

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

No branches or pull requests

2 participants