Repository navigation
fix(deps): bump redis from 7.0.1 to 8.1.0 - #5379
Conversation
Bumps [redis](https://github.com/redis/redis-py) from 7.0.1 to 8.1.0. - [Release notes](https://github.com/redis/redis-py/releases) - [Changelog](https://github.com/redis/redis-py/blob/master/CHANGES) - [Commits](redis/redis-py@v7.0.1...v8.1.0) --- updated-dependencies: - dependency-name: redis dependency-version: 8.1.0 dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
Fixed the Redis 8 typing failures in 17ebe12: remove obsolete async-command ignores, narrow the count-free RPOP result to its scalar response, and assert the expected byte result directly in the shared-client test. Runtime behavior and the redis>=7 requirement are unchanged. Validation: focused Redis tests passed with both Redis 8.1.0 and 7.0.1 (114 passed, 4 skipped each); independent review found no issues; the complete repository verification stack passed formatting, lint, Mypy, Pyright, 12,355 parallel tests and 107 serial tests (70 total skips). CI is running on the updated head. |
|
Codex Review: Didn't find any major issues. Can't wait for the next one! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
🛡️ Codex Security Review · Automatically triggeredSecurity review completed. No security issues were found in this pull request. Reviewed commit: Only the user who started this review can view the report in Codex. ℹ️ About Codex security reviews in GitHubThis is an experimental Codex feature. Security reviews are triggered when:
Once complete, Codex will leave suggestions, or a comment if no findings are found. |
jbeckwith-oai
left a comment
jbeckwith-oai
left a comment
There was a problem hiding this comment.
Reviewed the complete Redis 8.1.0 update and follow-up typing fix. The scalar RPOP cast matches the count-free command; obsolete async type ignores are removed, and the shared-client lifecycle assertion preserves the expected bytes contract. No runtime behavior or public dependency constraint changes. Independent review and fresh code/security reviews found no outstanding issues. All 23 CI checks pass on this head; local full verification and focused Redis 7/8 compatibility tests also pass.
marcuswood-oai
left a comment
marcuswood-oai
left a comment
There was a problem hiding this comment.
LGTM 👍 No changes requested.
Verified RedisSession behavior with redis-py 7.0.0 and 8.1.0, including real-server RESP2/RESP3, bytes/string responses, and client ownership. The typing changes preserve the existing redis>=7 contract. Redis 8's upstream connection and retry defaults still apply to environments using the updated lockfile.
markstuart-oai
left a comment
markstuart-oai
left a comment
There was a problem hiding this comment.
Reviewed 17ebe12. The local typing changes preserve the existing single-item RPOP, decoding, cancellation, and shared-client behavior. The cast matches the count-free command, and the revised test still checks that the shared client remains usable after session close. No actionable findings.
Hosted type checking, Python-version tests, and package checks passed on this head. This was a source-only review; I ran no repository workloads.
3 participants
Bumps redis from 7.0.1 to 8.1.0.
Release notes
Sourced from redis's releases.
... (truncated)
Commits
e013126Testing with 8.10 GA (#4227)059d1e9Fixed TS.NRANGE commands to correctly apply aggregators (#4225)d486a0aFix FIELDNAME alias dropping first character of un-prefixed fields (#4224)fb93104feat: add client-side HIMPORT fieldset support for standalone and cluster cli...9197609Fix lat/lon swap in search querystring geo() helper (#4223)14714a0fix: Fixed double decoding issue with unquote() (#4222)227280aUpdate lib version to 8.1.088d16d0Decode ACL LOG string values on the default RESP3 legacy callback (#4201)599fd75feat: add FT.ALIASLIST support (#4198)71e275etest: add VectorField RERANK serialization tests for sync and async search (#...Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore this major versionwill close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor versionwill close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependencywill close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)