Skip to content

fix: add lint script to project - #320

Open
Rajvardhan-Singh-Rathore wants to merge 1 commit into
AOSSIE-Org:mainfrom
Rajvardhan-Singh-Rathore:feature/add-lint-script
Open

Rajvardhan-Singh-Rathore wants to merge 1 commit into
AOSSIE-Org:mainfrom
Rajvardhan-Singh-Rathore:feature/add-lint-script

Conversation

@Rajvardhan-Singh-Rathore

@Rajvardhan-Singh-Rathore Rajvardhan-Singh-Rathore commented Oct 7, 2026 •

Copy link
Copy Markdown

Addressed Issues:

[FEATURE]: Add missing lint and format scripts to package.json
#82
Fixes #82

Additional Notes:

  • Added lint and lint:fix scripts to package.json.
  • Installed missing devDependency eslint@latest @eslint/js@latest and devDependencies required by eslint.config.js: globals, typescript-eslint, eslint-plugin-react-hooks, and eslint-plugin-react-refresh.
  • npm run lint (eslint .) - It will scan the code and print any errors or warnings.
  • npm run lint:fix (eslint . --fix) - It will automatically correct linting and syntax issues and nothing will change if everything is right.
  • I tested running npm run lint, npm run lint:fix and npm run build and all of them succeeded.
  • I have also updated "Quick Start" section in the docs (Readme.md).

Checklist

  • My code follows the project's code style and conventions
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings or errors
  • I have joined the Discord server and I will share a link to this PR with the project maintainers there
  • I have read the Contributing Guidelines

⚠️ AI Notice - Important!

We encourage contributors to use AI tools responsibly when creating Pull Requests. While AI can be a valuable aid, it is essential to ensure that your contributions meet the task requirements, build successfully, include relevant tests, and pass all linters. Submissions that do not meet these standards may be closed without warning to maintain the quality and integrity of the project. Please take the time to understand the changes you are proposing and their impact.

Summary by CodeRabbit

  • Documentation
    • Updated the Quick Start guide with instructions for checking code quality and automatically fixing lint issues. The development server and production build steps are renumbered.
  • Chores
    • Added commands for checking and fixing lint issues during development.

@coderabbitai

coderabbitai Bot commented Oct 7, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Repository: AOSSIE-Org/OrgExplorer/.coderabbit.yaml
  • Review profile: ASSERTIVE
  • Plan: Advanced
  • Run ID: cd1f4040-d5c2-4b20-991c-a683b2935ffa
📥 Commits

Reviewing files that changed from the base of the PR and between 87d3ebd and e575b9c.

⛔ Files ignored due to path filters (1)
  • package-lock.json is excluded by !**/package-lock.json
📒 Files selected for processing (2)
  • README.md
  • package.json

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.


Walkthrough

The package adds ESLint development dependencies and lint and lint:fix scripts. The README Quick Start documents both commands and renumbers the development-server instructions.

Changes

Lint Setup

Layer / File(s) Summary
Configure and document lint commands
package.json, README.md
package.json adds ESLint and related development dependencies, plus lint and lint:fix scripts. The Quick Start adds commands for both scripts and renumbers the development-server step.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~8 minutes

Change: Other · Severity of issue fixed: Low

Suggested labels: Documentation, Bash Lang

Suggested reviewers: rahul-vyas-dev

Merge Risk: ⚪ Minimal · up to e575b

The PR adds lint commands to the README. The verified formatting warning has no material user or operational impact, so no significant merge risk remains.

🚥 Pre-merge checks | ✅ 3 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Linked Issues check ⚠️ Warning Issue #82 requests both linting and formatting scripts, ESLint and Prettier dependencies, ESLint rules for React and Prettier compatibility, and a corresponding CONTRIBUTING.md checklist update. The P… Add the requested formatting script and Prettier dependencies, update CONTRIBUTING.md to reference the available scripts, and ensure the ESLint configuration includes the requested React rules and Prettier compatibility.
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly describes the main change: adding lint scripts to the project.
Out of Scope Changes check ✅ Passed The reported lint and lint:fix scripts, ESLint dependencies, and README Quick Start update support the linting and contributor-onboarding objective in issue #82. The summary shows no unrelated cha…
Full details: Linked Issues check

Explanation

Issue #82 requests both linting and formatting scripts, ESLint and Prettier dependencies, ESLint rules for React and Prettier compatibility, and a corresponding CONTRIBUTING.md checklist update. The PR summary reports lint and lint:fix scripts and ESLint-related dependencies, but no format script or Prettier dependencies. The summary reports a README.md update, not a CONTRIBUTING.md update. It does not establish whether the existing ESLint configuration meets the requested rule requirements.

  • Fix all pre-merge checks with AI
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create a new PR
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

A rabbit checks the linting run,
Then fixes errors one by one.
The Quick Start shows the way,
New scripts join the workday.
A hop, a build, and all is done!

Comment @coderabbitai help to get the list of available commands.

@github-actions github-actions Bot added enhancement New feature or request configuration Config file changes dependencies Dependency updates documentation Documentation updates javascript JavaScript/TypeScript changes size/XL 500+ lines changed first-time-contributor First time contributor labels Oct 7, 2026
@socket-security

Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Addedtypescript-eslint@​8.71.11001007498100
Addedglobals@​17.13.01001008695100
Added@​eslint/​js@​10.0.110010010087100
Addedeslint-plugin-react-refresh@​0.5.71001009992100
Addedeslint-plugin-react-hooks@​7.1.110010010096100
Addedeslint@​10.12.09710010097100

View full report

@github-actions github-actions Bot added size/XL 500+ lines changed and removed size/XL 500+ lines changed labels Oct 7, 2026

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

configuration Config file changes dependencies Dependency updates documentation Documentation updates enhancement New feature or request first-time-contributor First time contributor javascript JavaScript/TypeScript changes size/XL 500+ lines changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEATURE]: Add missing lint and format scripts to package.json

1 participant