Skip to content

Conversation

@railgun19457
Copy link
Contributor

@railgun19457 railgun19457 commented Nov 8, 2025

Added Docker-based one-click deployment instructions for AstrBot.

Sourcery 总结

文档:

  • 在 community-deployment.md 中记录了一个基于 Docker 的一键部署脚本,其中包含通过 curl 或 wget 使用的方法、sudo 权限提升、可选标志、镜像预览以及仓库链接。
Original summary in English

Summary by Sourcery

Documentation:

  • Document a Docker-based one-click deployment script in community-deployment.md with usage via curl or wget, sudo elevation, optional flags, image previews, and repository link

Added Docker-based one-click deployment instructions for AstrBot.
Copy link
Contributor

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

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

你好 - 我已审阅了你的更改 - 以下是一些反馈:

  • 通过调整标题(例如“基于Docker”与“原生部署”)来区分两个“Linux 一键部署脚本”部分,以避免混淆。
  • 添加一个先决条件部分,列出Docker(如果需要,还有Docker Compose),以确保用户在运行脚本前具备所需的环境。
  • 用标准的Markdown图像语法替换原始的HTML 标签,以保持文档风格一致。
AI代理提示
请处理此代码审查中的评论:

## 总体评论
- 通过调整标题(例如“基于Docker”与“原生部署”)来区分两个“Linux 一键部署脚本”部分,以避免混淆。
- 添加一个先决条件部分,列出Docker(如果需要,还有Docker Compose),以确保用户在运行脚本前具备所需的环境。
- 用标准的Markdown图像语法替换原始的HTML <img> 标签,以保持文档风格一致。

Sourcery 对开源项目免费 - 如果你喜欢我们的评论,请考虑分享它们 ✨
帮助我变得更有用!请点击每个评论上的 👍 或 👎,我将根据反馈改进你的评论。
Original comment in English

Hey there - I've reviewed your changes - here's some feedback:

  • Differentiate the two “Linux 一键部署脚本” sections by adjusting their headings (e.g. “基于Docker” vs “原生部署”) to avoid confusion.
  • Add a prerequisites section listing Docker (and Docker Compose if needed) to ensure users have the required environment before running the script.
  • Replace the raw HTML tags with standard Markdown image syntax to keep the document style consistent.
Prompt for AI Agents
Please address the comments from this code review:

## Overall Comments
- Differentiate the two “Linux 一键部署脚本” sections by adjusting their headings (e.g. “基于Docker” vs “原生部署”) to avoid confusion.
- Add a prerequisites section listing Docker (and Docker Compose if needed) to ensure users have the required environment before running the script.
- Replace the raw HTML <img> tags with standard Markdown image syntax to keep the document style consistent.

Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

Copy link
Contributor

@Sjshi763 Sjshi763 left a comment

Choose a reason for hiding this comment

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

用>导致分割了多行,个人感觉是不需要的

@Soulter
Copy link
Member

Soulter commented Nov 9, 2025

各种一键部署脚本建议放到社区部署那里 community-development.md

@railgun19457
Copy link
Contributor Author

这不就是吗👀

@Soulter
Copy link
Member

Soulter commented Nov 9, 2025

这不就是吗👀

看错了 抱歉

@Soulter Soulter merged commit 862307d into AstrBotDevs:v4 Nov 9, 2025
1 check passed
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.

3 participants