-
Notifications
You must be signed in to change notification settings - Fork 59.9k
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
Support saas version in readme #5529
Support saas version in readme #5529
Conversation
The latest updates on your projects. Learn more about Vercel for Git βοΈ
|
Caution Review failedThe pull request is closed. WalkthroughThe changes made in the pull request involve updates to the Changes
Possibly related PRs
Suggested reviewers
Poem
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? πͺ§ TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
The latest updates on your projects. Learn more about Vercel for Git βοΈ
|
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.
Actionable comments posted: 0
π§Ή Outside diff range and nitpick comments (2)
README.md (2)
15-20
: Consider the placement of the new NextChat-Saas badge.The addition of the NextChat-Saas badge is a good improvement to showcase the SaaS version. However, you might want to consider its placement in the list of badges. Typically, the most important or frequently used options are placed first. Is the SaaS version the primary offering, or should it be placed after the Web and desktop app badges?
π§° Tools
LanguageTool
[grammar] ~17-~17: The operating system from Apple is written βmacOSβ.
Context: ...[Windows][Windows-image]][download-url] [![MacOS][MacOS-image]][download-url] [![Linux][...(MAC_OS)
17-17
: Fix grammatical error in MacOS badge.There's a minor grammatical error in the MacOS badge. The correct spelling for Apple's operating system is "macOS" (lowercase 'm').
Apply this change to fix the error:
-[MacOS-image]: https://img.shields.io/badge/-MacOS-black?logo=apple +[macOS-image]: https://img.shields.io/badge/-macOS-black?logo=appleAlso, make sure to update any references to this badge throughout the document.
π§° Tools
LanguageTool
[grammar] ~17-~17: The operating system from Apple is written βmacOSβ.
Context: ...[Windows][Windows-image]][download-url] [![MacOS][MacOS-image]][download-url] [![Linux][...(MAC_OS)
π Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
π Files selected for processing (1)
- README.md (1 hunks)
π§° Additional context used
LanguageTool
README.md
[grammar] ~17-~17: The operating system from Apple is written βmacOSβ.
Context: ...[Windows][Windows-image]][download-url] [![MacOS][MacOS-image]][download-url] [![Linux][...(MAC_OS)
π Additional comments not posted (1)
README.md (1)
21-26
: LGTM: Updated links and badges.The addition of NextChatAI links and the update to the saas-url are consistent with the introduction of the NextChat-Saas feature. The new saas-image badge is also a good addition to visually represent the SaaS offering.
readme_cn readme_jpδΈθ΅·ζΉ |
readme_cn readme_jp change together |
Your build has completed! |
done~ |
done~ |
π» εζ΄η±»ε | Change Type
π εζ΄θ―΄ζ | Description of Change
support saas version
π θ‘₯ε δΏ‘ζ― | Additional Information
Summary by CodeRabbit
README.md
for improved accessibility of links.