Skip to content

chore: update Python version requirements to 3.12#4963

Merged
Soulter merged 1 commit intomasterfrom
chore/update-python-to-3.12
Feb 8, 2026
Merged

chore: update Python version requirements to 3.12#4963
Soulter merged 1 commit intomasterfrom
chore/update-python-to-3.12

Conversation

@Soulter
Copy link
Member

@Soulter Soulter commented Feb 8, 2026

Modifications / 改动点

  • This is NOT a breaking change. / 这不是一个破坏性变更。

Screenshots or Test Results / 运行截图或测试结果


Checklist / 检查清单

  • 😊 如果 PR 中有新加入的功能,已经通过 Issue / 邮件等方式和作者讨论过。/ If there are new features added in the PR, I have discussed it with the authors through issues/emails, etc.
  • 👀 我的更改经过了良好的测试,并已在上方提供了“验证步骤”和“运行截图”。/ My changes have been well-tested, and "Verification Steps" and "Screenshots" have been provided above.
  • 🤓 我确保没有引入新依赖库,或者引入了新依赖库的同时将其添加到了 requirements.txtpyproject.toml 文件相应位置。/ I have ensured that no new dependencies are introduced, OR if new dependencies are introduced, they have been added to the appropriate locations in requirements.txt and pyproject.toml.
  • 😮 我的更改没有引入恶意代码。/ My changes do not introduce malicious code.

由 Sourcery 提供的摘要

将项目的最低和运行时 Python 版本在容器和打包元数据中统一提升到 3.12。

新特性:

  • 为应用程序新增运行时依赖:python-socks

增强改进:

  • pyproject.toml 中的 requires-python 约束从 >=3.10 收紧为 >=3.12,以反映新的受支持运行时基线。

构建:

  • 将 Docker 基础镜像更新为 python:3.12-slim,并相应调整 .python-version
Original summary in English

Summary by Sourcery

Raise the project’s minimum and runtime Python version to 3.12 across container and packaging metadata.

New Features:

  • Add python-socks as a runtime dependency for the application.

Enhancements:

  • Tighten pyproject.toml requires-python constraint from >=3.10 to >=3.12 to reflect the new supported runtime baseline.

Build:

  • Update Docker base image to python:3.12-slim and align .python-version accordingly.

@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Feb 8, 2026
@dosubot
Copy link

dosubot bot commented Feb 8, 2026

Related Documentation

Checked 1 published document(s) in 1 knowledge base(s). No updates required.

How did I do? Any feedback?  Join Discord

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.

Hey - 我在这里给出了一些整体层面的反馈:

  • requires-python>=3.10 改为 >=3.12,会对使用 3.10/3.11 的用户造成破坏性变更;请确认这是有意为之,并且项目确实不再打算支持这些版本。
  • 随着在 dependencies 中新增 python-socks>=2.8.0,建议检查在 Dockerfile 中是否仍然需要单独显式安装 socksio,或者现在这两个库之间是否存在功能重叠和冗余。
供 AI Agent 使用的提示
Please address the comments from this code review:

## Overall Comments
- Changing `requires-python` from `>=3.10` to `>=3.12` is a breaking change for users on 3.10/3.11; please confirm this is intentional and that the project no longer aims to support those versions.
- With the addition of `python-socks>=2.8.0` to `dependencies`, consider verifying whether `socksio` is still needed as a separate explicit install in the Dockerfile or if there is now redundancy between these libraries.

Sourcery 对开源项目是免费的——如果你觉得我们的评审有帮助,欢迎分享给更多人 ✨
帮我变得更有用!请在每条评论上点击 👍 或 👎,我会根据你的反馈改进后续的评审。
Original comment in English

Hey - I've left some high level feedback:

  • Changing requires-python from >=3.10 to >=3.12 is a breaking change for users on 3.10/3.11; please confirm this is intentional and that the project no longer aims to support those versions.
  • With the addition of python-socks>=2.8.0 to dependencies, consider verifying whether socksio is still needed as a separate explicit install in the Dockerfile or if there is now redundancy between these libraries.
Prompt for AI Agents
Please address the comments from this code review:

## Overall Comments
- Changing `requires-python` from `>=3.10` to `>=3.12` is a breaking change for users on 3.10/3.11; please confirm this is intentional and that the project no longer aims to support those versions.
- With the addition of `python-socks>=2.8.0` to `dependencies`, consider verifying whether `socksio` is still needed as a separate explicit install in the Dockerfile or if there is now redundancy between these libraries.

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.

@dosubot dosubot bot added the area:core The bug / feature is about astrbot's core, backend label Feb 8, 2026
@Soulter Soulter merged commit 3fd8980 into master Feb 8, 2026
6 checks passed
@Soulter Soulter deleted the chore/update-python-to-3.12 branch February 8, 2026 15:13
@abc9878
Copy link

abc9878 commented Feb 8, 2026

改为3.12的话,openai-whisper就没法用了。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:core The bug / feature is about astrbot's core, backend size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants