Skip to content

[Doc] README updates#3

Merged
jiarong0907 merged 4 commits intomainfrom
doc/readme-update
Oct 26, 2025
Merged

[Doc] README updates#3
jiarong0907 merged 4 commits intomainfrom
doc/readme-update

Conversation

@jiarong0907
Copy link
Copy Markdown
Contributor

No description provided.

jiarong0907 and others added 4 commits October 26, 2025 00:18
Reorganized the README to include the Web Stack section before Project Structure and removed the old Web Stack section.
@jiarong0907
Copy link
Copy Markdown
Contributor Author

/gemini review

@jiarong0907 jiarong0907 merged commit bb3a791 into main Oct 26, 2025
2 checks passed
@jiarong0907 jiarong0907 deleted the doc/readme-update branch October 26, 2025 05:32
Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the README.md to simplify its content and add a 'For Developers' section. The changes generally improve the documentation by directing users to the website for detailed information and providing contribution guidelines. I've identified a couple of areas for improvement: the instructions for using the Gemini code reviewer could be clarified to avoid confusion, and several dependency versions in the 'Web Stack' section are inconsistent with the package-lock.json file. Updating these will enhance the accuracy and helpfulness of the README for new contributors.

Comment thread README.md
pre-commit run --all-files
```

We have set up Google Gemini to review your code. You can use it by running the following command:
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

The instruction for using Google Gemini for code review could be clearer. The command /gemini review is typically used as a comment in a pull request on platforms like GitHub, not run in a local terminal. This phrasing might confuse contributors.

Suggested change
We have set up Google Gemini to review your code. You can use it by running the following command:
We have set up Google Gemini to review your code. You can trigger it by adding the following comment to your pull request:

Comment thread README.md
Comment on lines +75 to +78
- **Frontend**: React 19.2.0 with TypeScript
- **Routing**: React Router DOM 6.8.1
- **Icons**: Lucide React 0.263.1
- **Charts**: Recharts 2.8.0
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

Some of the library versions listed in the 'Web Stack' section are out of sync with the package-lock.json file, which can be misleading for developers. Specifically:

  • React: 19.2.0 in README vs 18.3.1 in package-lock.json.
  • React Router DOM: 6.8.1 in README vs 6.30.1 in package-lock.json.
  • Recharts: 2.8.0 in README vs 2.15.4 in package-lock.json.

Please update these to reflect the actual versions used in the project.

Suggested change
- **Frontend**: React 19.2.0 with TypeScript
- **Routing**: React Router DOM 6.8.1
- **Icons**: Lucide React 0.263.1
- **Charts**: Recharts 2.8.0
- **Frontend**: React 18.3.1 with TypeScript
- **Routing**: React Router DOM 6.30.1
- **Icons**: Lucide React 0.263.1
- **Charts**: Recharts 2.15.4

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant