python-epo-ops-client icon indicating copy to clipboard operation
python-epo-ops-client copied to clipboard

type hints

Open antonkulaga opened this issue 1 year ago • 7 comments

Please, add typehints, it will make the library way easier to use!

antonkulaga avatar Dec 21 '23 21:12 antonkulaga

@antonkulaga PRs are always welcomed.

gsong avatar Dec 21 '23 22:12 gsong

Dear Anton,

thank you for writing in. Having type hints would certainly be an improvement, on both aspects of QA and UX. Would you have the capacity to submit a corresponding patch?

With kind regards, Andreas.

amotl avatar Jan 15 '24 19:01 amotl

It would be great to enhance the library with type hints, indeed. However since the library still supports python 3.6 the type annotations are quite limited. Python 3.7 allows the use of from __future__ import annotations (see docs) which would allow a more comprehensive set of type hints. Question: Is dropping support for python 3.6 an option for this library?

mattkeanny avatar Jan 24 '24 16:01 mattkeanny

It would be great to enhance the library with type hints, indeed. However since the library still supports python 3.6 the type annotations are quite limited. Python 3.7 allows the use of from __future__ import annotations (see docs) which would allow a more comprehensive set of type hints. Question: Is dropping support for python 3.6 an option for this library?

Seeing as Python 3.6 end-of-life happened on 2021-12-23, 3.7 EOL happened on 2023-06-27, I would say we can safely drop support for Python 3.6 and possibly even 3.7.

@mattkeanny Would you be up for making a PR with both type hints and for updating support for 3.8 and up only? This would be a major version bump.

gsong avatar Jan 24 '24 16:01 gsong

Great. Will look into that. Thanks for considering.

mattkeanny avatar Jan 24 '24 16:01 mattkeanny

What's the good news? I suspect ChatGPT could add type hints quite reliably... If nobody's working on it, I don't mind using CoPilot to make a PR...

dbolser avatar Mar 24 '24 20:03 dbolser