deps: update redis requirement from >=5.2.0 to >=7.4.0#50
Merged
Conversation
Updates the requirements on [redis](https://github.com/redis/redis-py) to permit the latest version. - [Release notes](https://github.com/redis/redis-py/releases) - [Changelog](https://github.com/redis/redis-py/blob/master/CHANGES) - [Commits](redis/redis-py@v5.2.0...v7.4.0) --- updated-dependencies: - dependency-name: redis dependency-version: 7.4.0 dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com>
|
Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits. |
LEEI1337
pushed a commit
that referenced
this pull request
May 25, 2026
Three fixes to restore green CI on main (red since 2026-04-13):
1. docker/Dockerfile: add `COPY README.md .` before pip install.
pyproject.toml has `readme = "README.md"`, so hatchling needs the
file in build context. Fixes: OSError: Readme file does not exist.
2. ruff lint debt: auto-fix 131 I001 import-order + 33 W293 whitespace
errors. Manual fixes for 6 real code issues:
- 4× B905 zip() → zip(..., strict=True)
- 2× UP042 (str, Enum) → enum.StrEnum
- 1× F841 unused result variable
- 1× B017 broad pytest.raises(Exception) → noqa with rationale
3. pyproject.toml ruff config: add E501 + B008 to ignore list.
E501 (23 pre-existing line-too-long) is cosmetic tech-debt.
B008 (FastAPI Depends/Security in defaults) is idiomatic for FastAPI
and ruff's own docs acknowledge the exception.
Tests: 209 passed locally with Python 3.12 + pytest 9.0.3.
Unblocks dependabot PRs #50-59 (W38-D analysis).
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Member
|
W46-A NSS-DEP-HEAL Triage (2026-05-26) MAJOR (redis 5→7). Lokale Test-Suite (209/209) grün mit redis 7.4.0. Kein API-Break entdeckt. CI-rot ausschließlich durch pre-existing main-Tech-Debt (siehe #60). Status: STAY-OPEN. Empfehlung: Erst #60 (main-CI healen) mergen, dann diesen PR. Lokale Verifikation: 2026-05-26 W46-A Sub-Agent. — Beleg: |
2 tasks
LEEI1337
added a commit
that referenced
this pull request
May 26, 2026
…P2) (#62) * fix(knowledge): migrate qdrant search() to query_points() for v1.18 (#60 P2a) qdrant-client >=1.13 removed the `search()` method. Production code raised `AttributeError: 'QdrantClient' has no attribute 'search'`; tests masked the bug via MagicMock. - src/nss/knowledge/vector_store.py: call query_points() and iterate the QueryResponse.points list. - tests/test_knowledge/test_vector_store.py: mock query_points returning a response object with a .points attribute. Verified locally: 209/209 pytest tests pass with qdrant-client 1.18.0, no behavior change in the search contract (id/score/payload dict shape). Refs: nss#60 (mypy attr-defined on vector_store.py:90) Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> * chore(types): resolve 26 remaining mypy errors blocking CI (#60 P2b) Heal `mypy --ignore-missing-imports` on main (red since W46). Per-file fixes: - src/nss/middleware.py — type `call_next: RequestResponseEndpoint` and `app: ASGIApp` (starlette types); removes 4× `no-any-return` from `await call_next()` and the `Any` ASGI app param. - src/nss/knowledge/sag_encryption.py — type-annotate decoded payload. - src/nss/knowledge/rag_pipeline.py — annotate `results` so the return type matches `list[dict[str, Any]]`. - src/nss/knowledge/embeddings.py — replace 2× now-stale `type: ignore[union-attr]` with explicit `list[float]`/`list[list[float]]` intermediates (no-any-return). - src/nss/cache.py — handle redis-py 7.4 `Awaitable[bool] | bool` return of `ping()` without awaiting a bool literal. - src/nss/auth.py — return-type for `require_role` factory: `Callable[..., Awaitable[dict[str, Any]]]`. - src/nss/guardian/sentinel.py — `float(...)` cast on cosine result. - src/nss/guardian/server.py — `lifespan(app) -> AsyncIterator[None]`. - src/nss/metrics_server.py — return type on `metrics()` / `metrics_prometheus()`, hoist `PlainTextResponse` import, and widen `kwargs: dict[str, Any]` for uvicorn.run (uvicorn's **kwargs is heterogeneous; `dict[str, str]` produced 11 false-positive arg-type errors against the uvicorn stubs). - src/nss/gateway/server.py — `list[Any]` for `_fire_dpia.entities`. Verified locally (Python 3.12, pytest 9.0.3 + all 10 Dependabot dep-bumps installed simultaneously): mypy = 0 errors, ruff = clean, 209/209 tests pass. Unblocks Dependabot PRs #50-#59. Refs: nss#60 (27 mypy errors). qdrant migration carved out into prior commit so this change is pure annotation work with no runtime delta. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> --------- Co-authored-by: mackbook <mackbook@mackbooks-MacBook-Air.local> Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Updates the requirements on redis to permit the latest version.
Release notes
Sourced from redis's releases.
Commits
b72f24aUpdating lib version to 7.4.00a4e0afRefactored health check logic for MultiDBClient (#3994)15492c9Refactored connection count and SCH metric collection (#4001)cd964acExpose basic Otel classes and funtions to be importable through redis.observa...46ab74dFixing security concern in repr methods for ConnectionPools - passwords m...26482dbFix AttributeError in cluster metrics recording when connection is None or Cl...8ecbc7aUpdating lib version to 7.3.011043dftyping: accept single-key input for blpop and brpop (#3990)d958125fix: use KeysT for blpop and brpop keys parameter type annotation (#3987)75bf91b[async] Adding access to cluster client's nodes_manager and set_response_call...You can trigger a rebase of this PR 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)