Fix: check for typeddict before class or subclass checks which fail

[z3z1ma]

Description

Just a small fix to ensure union types containing TypedDicts don't fail as they currently do.

Reproducible Example:

class NoopProviderOptions(t.TypedDict): ...


@t.overload
def load_feature_flag_provider(
    provider: None = None,
    options: t.Optional[NoopProviderOptions] = None,
) -> SupportsFFs: ...


@t.overload
def load_feature_flag_provider(
    provider: t.Literal["noop"] = "noop",
    options: t.Optional[NoopProviderOptions] = None,
) -> SupportsFFs: ...


class FileProviderOptions(t.TypedDict):
    path: str


@t.overload
def load_feature_flag_provider(
    provider: t.Literal["file"] = "file",
    options: t.Optional[FileProviderOptions] = None,
) -> SupportsFFs: ...


class HarnessProviderOptions(t.TypedDict):
    api_key: str
    sdk_key: str
    account: str
    organization: str
    project: str


@t.overload
def load_feature_flag_provider(
    provider: t.Literal["harness"] = "harness",
    options: t.Optional[HarnessProviderOptions] = None,
) -> SupportsFFs: ...


class LaunchDarklyProviderOptions(t.TypedDict):
    sdk_key: str


@t.overload
def load_feature_flag_provider(
    provider: t.Literal["launchdarkly"] = "launchdarkly",
    options: t.Optional[LaunchDarklyProviderOptions] = None,
) -> SupportsFFs: ...


@with_config(sections=("feature_flags",))
def load_feature_flag_provider(
    provider: t.Optional[t.Literal["file", "harness", "launchdarkly", "noop"]] = None,
    options: t.Optional[
        t.Union[
            NoopProviderOptions,
            FileProviderOptions,
            HarnessProviderOptions,
            LaunchDarklyProviderOptions,
        ]
    ] = None,
) -> SupportsFFs: ...

The load_feature_flag_provider will fail without my fix because you cannot do isclass or issubclass on a TypedDict...

Related Issues

• Fixes #...
• Closes #...
• Resolves #...

Additional Context

[netlify]

✅ Deploy Preview for dlt-hub-docs canceled.

Name	Link
🔨 Latest commit	b0bf9e0
🔍 Latest deploy log	https://app.netlify.com/sites/dlt-hub-docs/deploys/6605de3651fb1a00085ea852

[rudolfix]

LGTM!