You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository was archived by the owner on Jan 26, 2019. It is now read-only.
We currently link to this project from React documentation. I haven’t tried it personally but I assumed it would follow the philosophy in CRA. In particular, we don’t use lint warnings for things that are inconsequential (code style) and only use them to warn people about actual bugs in their code. This was a major design decision after seeing beginners struggle with screens fully of linter errors about spacing and alphabetising props when they’re just learning a library!
However, I recently realized this project doesn’t actually subscribe to this philosophy, and has very strict lint rules:
I don’t want to debate usefulness of this, but I’m wondering how intentional it was. If it is intentional I think we’ll need to avoid recommending this setup for beginners, for the same reason we don’t recommend Airbnb config to them. But maybe it’s not that important and you can consider relaxing them to stay closer to CRA’s original vision?
Thanks for consideration.
johannes-scharlach, NMinhNguyen, kevin940726, OrderAndCh4oS, lionralfs and 63 moreacro5piano, danywu, gnapse, sebastienjust, mikehibm and 1 morecesaraaron, Markus-ipse, bsalesc, dtinth, gnapse and 7 more