Yes, sure, sorry for not explaining it from the start.
So, knowing that the official getUpdates API declares timeout as Integer, here are some of the inconsistencies:
-
The
Updater.start_pollingcorrectly declares it asint:timeout: int = 10, but it's incorrectly documented as
float:timeout (:obj:`float`, optional): Passed to -
The
Bot.get_updatesincorrectly declares it asfloat:timeout: Optional[float] = None, although it's correctly documented as
int:timeout (:obj:`int`, optional): Timeout in seconds for long polling. Defaults to ``0``,
So, the bottom line is that all of them should be int, which is what this PR does.