-
Notifications
You must be signed in to change notification settings - Fork 124
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ADD Eslint rules for the app #436
Comments
Looks good to me after taking a glance at the ESLint. |
I can't seem to figure out how ESLint would exactly work as a plugin🤔 Would it show us the errors inside the IDE or how? Instead what I suggest is to use a linter that would run as a GitHub Action workflow. I think https:/gaurav-nelson/github-action-markdown-link-check would be a good option if we are to go for this idea. WDYT? @RainbowMango But obviously, if there are some particular use cases for ESLint that needs to have a check for, we can integrate these both too. |
@rakshitgondwal @RainbowMango Integrating ESLint in our code will help us keep track of the incoming Pull Requests in the future and maintain good coding practices if we add it as a GitHub workflow. |
What would you like to be added:
To maintain standardized coding styles let us add eslint configuration
Why is this needed:
To maintain uniformity in the code
https://docusaurus.io/docs/api/misc/@docusaurus/eslint-plugin
The text was updated successfully, but these errors were encountered: