Update package.md#6293
Conversation
Summary of ChangesHello, 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 updates the Chinese deployment documentation for Highlights
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.
Code Review
This pull request updates the command to astrbot run in the Chinese documentation. I've provided a suggestion to further improve the clarity of the instructions. Additionally, I noticed that the corresponding English documentation in docs/en/deploy/astrbot/package.md still contains the outdated astrbot command and is missing the astrbot init step. It would be beneficial to update the English documentation as well to ensure consistency.
| ```bash | ||
| uv tool install astrbot | ||
| astrbot init # 只需要在第一次部署时执行,后续启动不需要执行 | ||
| astrbot | ||
| astrbot run | ||
| ``` |
There was a problem hiding this comment.
For better clarity, consider separating the commands for the first-time setup and subsequent runs. The current inline comment is helpful, but explicitly structuring the document with separate sections for 'First-time deployment' and 'Subsequent starts' would make it easier for users to follow.
For example:
首次部署 (First-time deployment)
uv tool install astrbot
astrbot init
astrbot run后续启动 (Subsequent starts)
astrbot runThere was a problem hiding this comment.
Pull request overview
Updates the Chinese uv-based deployment guide to use the correct CLI subcommand for starting AstrBot after initialization, aligning the documentation with the current astrbot CLI behavior.
Changes:
- Replace the outdated
astrbotstart command withastrbot runin the uv deployment instructions. - Keep the initialization step (
astrbot init) explicitly documented as first-run only.
You can also share your feedback on Copilot code review. Take the survey.
Modifications / 改动点
Screenshots or Test Results / 运行截图或测试结果
Checklist / 检查清单
requirements.txt和pyproject.toml文件相应位置。/ I have ensured that no new dependencies are introduced, OR if new dependencies are introduced, they have been added to the appropriate locations inrequirements.txtandpyproject.toml.Summary by Sourcery
Documentation:
astrbot runinstead of the previousastrbotcommand.