Add more type annotations

jaraco / keyring

MIT License

1.24k stars 152 forks source link

Closed lovetox closed 1 year ago

lovetox commented 1 year ago

Hi,

I tried to add more type annotations

The docs say priority must be defined and is a property and must return float or int, but in the base class it was a method returning None. I tried to make this more consistent across the backends to make type checkers happy.

jaraco commented 1 year ago

I've rebased the changes on top of 687d2c5 (fixing the emergent black failure).

lovetox commented 1 year ago

I remove the commit, so everything is ready from my side