Skip to content

fix: update Python version constraints in project files#129

Merged
CaralHsi merged 3 commits intoMemTensor:devfrom
Ki-Seki:feat/py-ver
Jul 21, 2025
Merged

fix: update Python version constraints in project files#129
CaralHsi merged 3 commits intoMemTensor:devfrom
Ki-Seki:feat/py-ver

Conversation

@Ki-Seki
Copy link
Member

@Ki-Seki Ki-Seki commented Jul 19, 2025

Description

Summary:

647cd098954c7147ac4d7adee34ffa1d

According to @likeUMR, the current Python version requirement in pyproject.toml is >=3.10,<4.0 (too restrictive), which sometimes causes Poetry to fail version resolution when conducting poetry add MemoryOS.

The current solution follows Poetry's official recommendation: https://python-poetry.org/docs/pyproject#requires-python.

It separates the more lenient project section (as per Python PEP 621) from Poetry's stricter tool.poetry.dependencies (see: Poetry documentation). This approach satisfies both cases: managing poetry add MemoryOS (requiring looser Python constraints) and adding dependencies for MemoryOS (requiring more precise Python constraints).

Reviewer: @CaralHsi @likeUMR

Checklist:

  • I have performed a self-review of my own code | 我已自行检查了自己的代码
  • I have mentioned the person who will review this PR | 我已提及将审查此 PR 的人

@Ki-Seki Ki-Seki requested review from CaralHsi and likeUMR July 20, 2025 13:58
@Ki-Seki Ki-Seki marked this pull request as ready for review July 20, 2025 14:21
Copilot AI review requested due to automatic review settings July 20, 2025 14:21

This comment was marked as resolved.

Copy link
Collaborator

@CaralHsi CaralHsi left a comment

Choose a reason for hiding this comment

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

Looks Good

@CaralHsi CaralHsi merged commit 04e7dda into MemTensor:dev Jul 21, 2025
20 checks 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.

2 participants