From 7464d63596f8b6fd993315a67c751d703d659305 Mon Sep 17 00:00:00 2001 From: Philip Bjorge Date: Wed, 13 Sep 2023 16:06:07 -0700 Subject: [PATCH] chore: remove __getattr__ from container --- src/dependency_injector/containers.pyi | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/dependency_injector/containers.pyi b/src/dependency_injector/containers.pyi index e773bfb3..3bad212e 100644 --- a/src/dependency_injector/containers.pyi +++ b/src/dependency_injector/containers.pyi @@ -46,7 +46,9 @@ class Container: def __init__(self) -> None: ... def __deepcopy__(self, memo: Optional[Dict[str, Any]]) -> Provider: ... def __setattr__(self, name: str, value: Union[Provider, Any]) -> None: ... - def __getattr__(self, name: str) -> Provider: ... + # While __getattr__ is technically on the public interface, we should never use it + # since we're utilizing DeclarativeContainers + # def __getattr__(self, name: str) -> Provider: ... def __delattr__(self, name: str) -> None: ... def set_providers(self, **providers: Provider): ... def set_provider(self, name: str, provider: Provider) -> None: ...