Fix for Property, add explicit hint for signal in notify argument

[kaneryu]

No description provided.

[kaneryu]

I've been getting hundreds of these errors:

Argument "notify" to "Property" has incompatible type "Signal"; expected "Callable[..., Any] | None"mypy[arg-type](https://mypy.readthedocs.io/en/stable/_refs.html#code-arg-type)

(variable) urlChanged: Signal

And this fix resolves them.

[bluebird75]

Hi,

Thanks for the contribution.

Could you also provide a simple example of failing code ?

[kaneryu]

Can you please clarify what you mean by failing?
In the example above, even though I am using the proper method to create a signal and assign it to the property, mypy is still giving me a type error. In my opinion, that is the example of failing code.

[matiaslina]

Hi! I'm experiencing this same issue.

Here is a minimal example that shows this error:

from PySide6.QtCore import QObject, Property, Signal


class Test(QObject):
    testChanged = Signal()

    def get_test(self) -> str:
        return 'test'

    def set_test(self, str) -> None:
        pass

    test = Property(str, get_test, set_test, notify=testChanged)

mypy t.py
t.py:13: error: Argument "notify" to "Property" has incompatible type "Signal"; expected "Callable[..., Any] | None"  [arg-type]
Found 1 error in 1 file (checked 1 source file)