Skip to content

Allow redis client 8 releases - #854

Open
abrookins wants to merge 3 commits into
mainfrom
fix/deps-redis-upper-bound
Open

Allow redis client 8 releases#854
abrookins wants to merge 3 commits into
mainfrom
fix/deps-redis-upper-bound

Conversation

@abrookins

@abrookins abrookins commented Aug 1, 2026

Copy link
Copy Markdown
Collaborator

Supersedes #838. Updates the direct requirement and the corresponding uv.lock metadata.


Note

Low Risk
Dependency constraint-only change with no runtime code modifications; main risk is potential incompatibility with redis-py 8.x APIs that consumers may hit outside this repo’s pinned tests.

Overview
Widens the redis dependency ceiling from <8.0.0 to <9.0.0 in pyproject.toml, so installs can resolve redis-py 8.x while still excluding 9.0+.

The matching uv.lock project metadata is updated to the same specifier; no library or test code changes.

Reviewed by Cursor Bugbot for commit a6f0493. Bugbot is set up for automated code reviews on this repo. Configure here.

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit f94cad6. Configure here.

Comment thread pyproject.toml
]
dependencies = [
"redis>=4.2.0,<8.0.0",
"redis>=4.2.0,<9.0.0",

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Redis 8 still blocked by redisvl

Medium Severity

Widening the direct redis upper bound to &lt;9.0.0 does not allow redis-py 8. The required dependency redisvl (&gt;=0.16.0, including current releases) still requires redis&lt;8.0, so resolvers keep selecting redis 7.x—and pinning redis&gt;=8 fails. The declared change overstates redis client 8 support.

Additional Locations (2)
Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit f94cad6. Configure here.

@abrookins

Copy link
Copy Markdown
Collaborator Author

Blocked by redis/redis-vl-python#686 for now

@abrookins

Copy link
Copy Markdown
Collaborator Author

Opened a PR upstream redis/redis-vl-python#695

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.

1 participant