python-mocket icon indicating copy to clipboard operation
python-mocket copied to clipboard

Add type hints to improve codebase readability and enable static type checking

Open Niraj-Kamdar opened this issue 1 year ago • 8 comments

I propose adding type hints to the mocket Python package to enhance code readability and maintainability. Type hints provide several benefits, including improved developer productivity, enhanced code documentation, and the ability to catch potential errors using static type-checking tools like Mypy.

Niraj-Kamdar avatar Jun 28 '23 06:06 Niraj-Kamdar

Hi @Niraj-Kamdar, I am not sure if I got your proposal right. Are you volunteering to add type hints to the project?

mindflayer avatar Jul 01 '23 14:07 mindflayer

No answers to my question, closing it.

mindflayer avatar Jul 06 '23 05:07 mindflayer

Hey @mindflayer Sorry for the late reply here. My proposal is for adding type hints in general, I would love to help here though when I get free time but currently, I have too much on my plate to help out with this. I would appreciate if you can keep this issue open so others or me can eventually take this and add type-hints to the mocket.

Niraj-Kamdar avatar Jul 11 '23 19:07 Niraj-Kamdar

@mindflayer Closing this issue signals that

  1. You don't plan to add type annotations.
  2. You aren't willing to accept contributions that add these.

Can you re-open this issue if one or both of these are untrue?

Kilo59 avatar Apr 25 '24 19:04 Kilo59

@mindflayer Closing this issue signals that

1. You don't plan to add type annotations.

2. You aren't willing to accept contributions that add these.

Can you re-open this issue if one or both of these are untrue?

I am the only maintainer of Mocket, and I think it's worth keeping non-bug issues open only if I am confident it's something I have time for, or at least if I know someone is willing to help with. Having a backlog of stuff just for the sake of it it's pointless. So, allow me to bounce back the question: is anyone willing to contribute with adding type hints?

mindflayer avatar Apr 25 '24 19:04 mindflayer

So, allow me to bounce back the question: is anyone willing to contribute with adding type hints?

@mindflayer

Yes. The only thing that gives me pause about taking on this work is the fact that this issue was closed. Which makes it ambiguous as to whether an annotation PR would be accepted.

Kilo59 avatar Apr 25 '24 21:04 Kilo59

Well, alright then! 👍

mindflayer avatar Apr 26 '24 05:04 mindflayer

Hi @Kilo59, are you still planning to continue with this task?

mindflayer avatar Sep 06 '24 11:09 mindflayer