-
-
Notifications
You must be signed in to change notification settings - Fork 1k
Add venv volume to compose.yml #3523
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: master
Are you sure you want to change the base?
Conversation
映射venv环境文件,避免重复下载依赖
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.
嘿,你好!我已经审阅了你的更改,它们看起来很棒!
帮助我更有用!请在每条评论上点击 👍 或 👎,我将使用这些反馈来改进你的评论。
Original comment in English
Hey there - I've reviewed your changes and they look great!
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
LIghtJUNction
left a comment
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.
很好
|
我个人用的方法是映射这个 随着python更新,venv可能变动,直接映射或许不是一个好主意。 个人暂时没有更新Astrbot。目前使用4.5.0。此意见仅供参考。 以上仅为个人意见和建议。 |
回复:
|
我在担心的是: 所以我在担心,未来Astrbot依赖的Python版本更新后,遗留的先前venv是否会阻碍其正常更新与运行。
Caching |

映射venv环境文件,避免重复下载依赖
fixes #XYZ
Motivation / 动机
Modifications / 改动点
Verification Steps / 验证步骤
Screenshots or Test Results / 运行截图或测试结果
Compatibility & Breaking Changes / 兼容性与破坏性变更
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.Sourcery 总结
改进:
./venv目录挂载到compose.yml中的/AstrBot/venv,以在容器重启之间缓存虚拟环境Original summary in English
Summary by Sourcery
Enhancements: