refactor: drop Enum patch; raise TypeError when @deprecated is applied to a class

[Borda]

Before submitting

• Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
• Did you make sure to update the docs?
• Did you write any new necessary tests?

What does this PR do?

@deprecated was designed for callables (functions/methods). Support for Enum classes was added in #104 as a Python 3.12 compatibility patch via _DeprecatedEnumWrapper, not by design. That workaround had a known flaw: isinstance() checks silently broke because the class binding was replaced by a wrapper object. Now that @deprecated_class() exists as the proper API for class-level deprecation, the patch is removed.

Changes:

• packing() raises TypeError for any class source (Enum, dataclass, plain class), with a message pointing to @deprecated_class()
• Delete _DeprecatedEnumWrapper, _is_enum_value_case, _convert_enum_value_args, and ENUM_VALUE_PARAM
• Simplify _prepare_target_call: drop source_is_class / source_has_var_positional params, return Callable not tuple
• Migrate 9 @deprecated-on-class wrappers in the test collection to @deprecated_class() (7 Enums + 2 dataclasses)
• Replace TestIsEnumValueCase / TestConvertEnumValueArgs unit tests with TestDeprecatedClassGuard covering the new TypeError behaviour
• Update TestDeprecatedEnums to reflect proxy warning-on-attribute-access semantics (consistent with _DeprecatedProxy design)

Part of #109

PR review

Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.

Did you have fun?

Make sure you had fun coding 🙃

[dosubot]

Related Documentation

Checked 1 published document(s) in 1 knowledge base(s). No updates required.

^{How did I do? Any feedback?}  

[Copilot]

Pull request overview

This PR removes the historical Enum-class workaround in @deprecated and makes class decoration an explicit error, pushing class-level deprecation to the dedicated @deprecated_class() proxy API.

Changes:

• @deprecated now raises TypeError when applied to any class (Enum/dataclass/plain class), pointing users to @deprecated_class().
• Removes the Enum wrapper/Enum-special-casing helpers and simplifies _prepare_target_call.
• Updates test fixtures and integration/unit tests to use @deprecated_class() for Enums/dataclasses and to reflect proxy warning semantics.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.

File	Description
src/deprecate/deprecation.py	Removes Enum wrapper patch, adds class-guard TypeError, refactors _prepare_target_call and call path
tests/collection_deprecate.py	Migrates deprecated Enum/dataclass fixtures from @deprecated to @deprecated_class (incl. self-mapped enum)
tests/integration/test_classes.py	Updates enum integration tests to match _DeprecatedProxy warning-on-access behavior and new fixtures
tests/unittests/test_deprecation.py	Replaces Enum helper unit tests with coverage for the new “class guard” TypeError behavior

[Copilot]

Pull request overview

Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.

[Copilot]

Pull request overview

Copilot reviewed 8 out of 8 changed files in this pull request and generated 3 comments.

Comments suppressed due to low confidence (1)

src/deprecate/deprecation.py:521

• The TypeError message for applying @deprecated to a class points users to "@deprecated_class() from deprecate.proxy". Since deprecated_class is re-exported at the top-level (deprecate.deprecated_class), consider referencing the public import path (e.g., "use deprecate.deprecated_class" / "from deprecate import deprecated_class") to avoid steering users toward an internal submodule path.

        if inspect.isclass(source):
            raise TypeError(
                f"Cannot apply @deprecated to class '{source.__name__}'. "
                "For class-level deprecation use @deprecated_class() from deprecate.proxy."
            )

[Copilot]

Pull request overview

Copilot reviewed 8 out of 8 changed files in this pull request and generated 1 comment.