-
Notifications
You must be signed in to change notification settings - Fork 2
Fix minor grammatical errors and add inclusionAI info in README.md #5
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
Reviewer's guide (collapsed on small PRs)Reviewer's GuideUpdates README wording for clarity and neutrality and adds a new Artificial Intelligence project (inclusionAI) to the listed open-source initiatives. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
Summary of ChangesHello @xiaoya-yaya, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request enhances the Highlights
🧠 New Feature in Public Preview: You can now enable Memory to help Gemini Code Assist learn from your team's feedback. This makes future code reviews more consistent and personalized to your project's style. Click here to enable Memory in your admin console. Changelog
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
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.
Code Review
This pull request makes a minor grammatical correction and adds the inclusionAI project to the README.md. The changes are good, but for consistency, the corresponding README_CN.md file should also be updated to include the new project information.
| - [Eggjs](https://github.com/eggjs): A web framework's framework for Node.js. | ||
|
|
||
| ## Artificial Intelligence | ||
| - ✨[inclusionAI](https://github.com/inclusionAI): Dedicated towards advancing AGI, including LLMs, RL and agentic systems, and other AGI-related frameworks or applications. |
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.
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.
Hey - I've found 2 issues, and left some high level feedback:
- The new inclusionAI bullet uses an emoji where the surrounding items do not; consider removing it or aligning the style with the rest of the list for consistency.
- Before merging, please confirm that inclusionAI is an official Ant Group–related project and, if so, consider matching the description style to the other bullets (short, product-focused phrases rather than a full sentence).
Prompt for AI Agents
Please address the comments from this code review:
## Overall Comments
- The new inclusionAI bullet uses an emoji where the surrounding items do not; consider removing it or aligning the style with the rest of the list for consistency.
- Before merging, please confirm that inclusionAI is an official Ant Group–related project and, if so, consider matching the description style to the other bullets (short, product-focused phrases rather than a full sentence).
## Individual Comments
### Comment 1
<location> `profile/README.md:7` </location>
<code_context>
Ant Group offers a high value on open-source communities too. According to our vision, "Create future-ready digital infrastructure for the service industry, and bring small and beautiful changes to the world. " Also, we aspire to make small contributions and changes to open-source communities.
-The Ant Group is committed to open source and has released numerous kinds of projects, which are included below.
</code_context>
<issue_to_address>
**suggestion (typo):** Consider rephrasing "offers a high value on" to a more idiomatic expression like "places a high value on."
"Offers a high value on" is nonstandard English. Rephrasing to something like "Ant Group places a high value on open-source communities" keeps the meaning but reads more naturally.
```suggestion
Ant Group places a high value on open-source communities too. According to our vision, "Create future-ready digital infrastructure for the service industry, and bring small and beautiful changes to the world. " Also, we aspire to make small contributions and changes to open-source communities.
```
</issue_to_address>
### Comment 2
<location> `profile/README.md:18` </location>
<code_context>
- [Eggjs](https://github.com/eggjs): A web framework's framework for Node.js.
## Artificial Intelligence
+- ✨[inclusionAI](https://github.com/inclusionAI): Dedicated towards advancing AGI, including LLMs, RL and agentic systems, and other AGI-related frameworks or applications.
- [DB-GPT](https://github.com/eosphoros-ai): Building Open AI-Native Data Infrastructure.
- [CodeFuse](https://github.com/codefuse-ai): Develop Code Large Language Models (Code LLMs) to support and enhance full-lifecycle AI native software development.
</code_context>
<issue_to_address>
**suggestion (typo):** Replace "Dedicated towards" with "Dedicated to" for more natural phrasing.
"Dedicated to advancing AGI" is the more standard and natural phrasing in English here.
```suggestion
- ✨[inclusionAI](https://github.com/inclusionAI): Dedicated to advancing AGI, including LLMs, RL and agentic systems, and other AGI-related frameworks or applications.
```
</issue_to_address>Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
| @@ -6,7 +6,7 @@ Ant Group, the leading supplier of online financial services in China, has been | |||
|
|
|||
| Ant Group offers a high value on open-source communities too. According to our vision, "Create future-ready digital infrastructure for the service industry, and bring small and beautiful changes to the world. " Also, we aspire to make small contributions and changes to open-source communities. | |||
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.
suggestion (typo): Consider rephrasing "offers a high value on" to a more idiomatic expression like "places a high value on."
"Offers a high value on" is nonstandard English. Rephrasing to something like "Ant Group places a high value on open-source communities" keeps the meaning but reads more naturally.
| Ant Group offers a high value on open-source communities too. According to our vision, "Create future-ready digital infrastructure for the service industry, and bring small and beautiful changes to the world. " Also, we aspire to make small contributions and changes to open-source communities. | |
| Ant Group places a high value on open-source communities too. According to our vision, "Create future-ready digital infrastructure for the service industry, and bring small and beautiful changes to the world. " Also, we aspire to make small contributions and changes to open-source communities. |
| - [Eggjs](https://github.com/eggjs): A web framework's framework for Node.js. | ||
|
|
||
| ## Artificial Intelligence | ||
| - ✨[inclusionAI](https://github.com/inclusionAI): Dedicated towards advancing AGI, including LLMs, RL and agentic systems, and other AGI-related frameworks or applications. |
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.
suggestion (typo): Replace "Dedicated towards" with "Dedicated to" for more natural phrasing.
"Dedicated to advancing AGI" is the more standard and natural phrasing in English here.
| - ✨[inclusionAI](https://github.com/inclusionAI): Dedicated towards advancing AGI, including LLMs, RL and agentic systems, and other AGI-related frameworks or applications. | |
| - ✨[inclusionAI](https://github.com/inclusionAI): Dedicated to advancing AGI, including LLMs, RL and agentic systems, and other AGI-related frameworks or applications. |
Summary by Sourcery
Update the profile README to improve wording around Ant Group's open source commitment and highlight the inclusionAI project in the Artificial Intelligence section.
Documentation: