From 0a13d2c653142d356ba0152abd079c0f8b44dd40 Mon Sep 17 00:00:00 2001 From: Alex Dutton Date: Thu, 26 May 2022 10:56:54 +0100 Subject: [PATCH] Fix AlterModelManagers managers annotation to be [(str, Manager), ...] (#966) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Django generates AlterModelManagers with Sequence[Tuple[str, Manager]] arguments — not Sequence[Manager] — as shown by their tests[^1]. [^1]: https://github.com/django/django/blob/4.0.4/tests/migrations/test_operations.py#L3792-L3799 Closes #965. --- django-stubs/db/migrations/operations/models.pyi | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/django-stubs/db/migrations/operations/models.pyi b/django-stubs/db/migrations/operations/models.pyi index d9e1208..1381cc3 100644 --- a/django-stubs/db/migrations/operations/models.pyi +++ b/django-stubs/db/migrations/operations/models.pyi @@ -88,8 +88,8 @@ class AlterModelOptions(ModelOptionOperation): def __init__(self, name: str, options: Dict[str, Any]) -> None: ... class AlterModelManagers(ModelOptionOperation): - managers: Sequence[Manager] = ... - def __init__(self, name: str, managers: Sequence[Manager]) -> None: ... + managers: Sequence[Tuple[str, Manager]] = ... + def __init__(self, name: str, managers: Sequence[Tuple[str, Manager]]) -> None: ... class IndexOperation(Operation): option_name: str = ...