Hey man, this looks good to me, but there should be definitly someone else to look over this cough cough @Eldinnie? cough cough. Can you remove the #Todo line though and either put it in this conversation or in the related issue? Then codacy should be fine with it.

I've addressed the todo, should I squash the commits?
Sorry for the trouble, I wanted to help you a bit but It's my first time doing a PR in a big repo 😅.

No need to, we will before merging into master. Just go ahead and make as many commits as you want :)

@SnowyCoder Now you have the chance to get the true big repo experience™️ 😉
It would be great to have a test for that last change you made regarding the removal of the timeout job. Basically a test that demonstrates that the timeout job will be removed and not executed, even if you have a child handler that starts before the timeout would run out and ends after.

Refer to https://github.com/python-telegram-bot/python-telegram-bot/blob/master/tests/test_conversationhandler.py line 414 and following for inspiration.

I've been studying the test code and I noticed something strange: the assignment at line 494 (that has moved to 552 in the master branch) is duplicated, it seems like a merge error or a typo.

@SnowyCoder Interesting find ^^
Edit: e75615c introduced it, not quite sure what was going on here...

Thanks a lot for your contribution @SnowyCoder!

Thank you for the support!