deprecate_kwarg_name is not type aware

[thelittlebug]

thats the reason why you get no autocompletion for simple things like the following

Workplane().text("You rock", 12, 5,). #at this point the editor does not know how to autocomplete

but there is a simple solution to this issue:

TCallable = TypeVar("TCallable", bound=Callable)

class deprecate_kwarg_name:
    def __init__(self, name, new_name):
        self.name = name
        self.new_name = new_name

    def __call__(self, f: TCallable) -> TCallable:
        @wraps(f)
        def wrapped(*args, **kwargs):

            if self.name in kwargs:
                warn(
                    f"Kwarg <{self.name}> will be removed. Please use <{self.new_name}>",
                    FutureWarning,
                )

            return f(*args, **kwargs)

        return cast(TCallable, wrapped)

[adam-urbanczyk]

Could you open a PR?

[thelittlebug]

yes, i will definitely do this. sry, i hadn't the possibilities for that while i was posting this. 1-2 days and i'm in my used to work area ;)

[voneiden]

@thelittlebug I've opened PR #1348 on your behalf, you OK with that?