Adding eslint to project #2539
illiteratewriter
started this conversation in
Ideas
Replies: 2 comments 3 replies
-
@illiteratewriter : For migrating to the |
Beta Was this translation helpful? Give feedback.
3 replies
-
Currently, ESLint runs only on |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
Reactstrap earlier had eslint with the following config. But currently we do not have any. Also we have migrated from
airbnb
config to thereact-app
config. I'm not sure if there was any reason for the switch.I was thinking of going back to
airbnb
as it enforces rules such as single quotes.I ran the linter using
airbnb
and the old config as above and we have around 1267 errors. A few of them such asMust use destructuring props assignment
can be disabled and we can bring down the errors. The rest also is I think manageable. Let me know your thoughts on this.Beta Was this translation helpful? Give feedback.
All reactions