-
Notifications
You must be signed in to change notification settings - Fork 105
feat: add ESLint & Prettier configuration #805
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
base: main
Are you sure you want to change the base?
Conversation
@sheikhlimon is attempting to deploy a commit to the recode Team on Vercel. A member of the Team first needs to authorize it. |
Thank you for submitting your pull request! 🙌 We'll review it as soon as possible. The estimated time for response is 5–8 hrs. In the meantime, please provide all necessary screenshots and make sure you run - npm build run , command and provide a screenshot, a video recording, or an image of the update you made below, which helps speed up the review and assignment. If you have questions, reach out to LinkedIn. Your contributions are highly appreciated!😊 Note: I maintain the repo issue every day twice at 8:00 AM IST and 9:00 PM IST. If your PR goes stale for more than one day, you can tag and comment on this same issue by tagging @sanjay-kv. We are here to help you on this journey of open source. Consistent 20 contributions are eligible for sponsorship 💰 🎁 check our list of amazing people we sponsored so far: GitHub Sponsorship. ✨ 📚Your perks for contribution to this community 👇🏻
If there are any specific instructions or feedback regarding your PR, we'll provide them here. Thanks again for your contribution! 😊 |
Synced data from Linked IssuesLabels:
Assignees:
Milestones:
|
@sanjay-kv , need a look over here and may be @iitzIrFan can look . |
Thanks for reviewing this! Let me know if you’d like me to tweak anything. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@sheikhlimon I tried running the Eslint as follows,
npm run lint
npm run lint:fix
npm run format
and as per our codebase it caught impressive 180 changes which, cannot be directly released in prod. So was thinking that, it would be great if, you make a new PR after running Eslint to be reviewed and merged.
For this you can cherry pick from this PR to get those Eslint tests !

done @iitzIrFan |
Description
This PR adds ESLint and Prettier configuration to the project to ensure consistent code formatting and linting standards across all contributors. This setup provides automated code quality checks while remaining contributor-friendly with minimal, non-intrusive rules.
Fixes #787
Type of Change
Changes Made
Configuration Files Added
.eslintrc.cjs
- ESLint configuration for JavaScript/TypeScript with React support.prettierrc
- Prettier formatting configuration.prettierignore
- Excludes documentation files from formattingScripts Added to
package.json
npm run lint
- Check for linting issuesnpm run lint:fix
- Auto-fix linting issuesnpm run format
- Format code files (JS/TS/JSON/CSS only)npm run format:check
- Check formatting without modifying filesPhilosophy
This configuration prioritizes:
Dependencies
Testing Done
npm install
- All dependencies install correctlynpm run lint
- Linting works without errorsnpm run format
- Formatting works without breaking codenpm run build
- Build succeeds with new configuration.prettierignore
prevents MDX files from being corruptedFor Reviewers
This PR only adds configuration files. No code has been reformatted - that will be done in a separate PR after this is merged to keep the diff reviewable.
For Contributors
After this PR is merged, contributors should run before committing:
npm run lint:fix && npm run format
Or set up VS Code with ESLint and Prettier extensions for automatic formatting on save
Instruction will be added in community/understand-lint-check.md
Checklist
npm run build
and attached screenshot(s) in this PR.