Skip to content

Conversation

@Bibo-Joshi
Copy link
Member

@Bibo-Joshi Bibo-Joshi commented May 12, 2022

renaming of tg.bot_api_version to tg.__bot_api_version__ is just a suggestion so far. Apparently in the first discussion to add this variable we decided against that. I have changed my view on that, but that doesn't necessarily mean that we have to do the renaming :)

  • Add documentation for __version__, __version_info__ and bot api version
  • add unit tests for Version

@Bibo-Joshi Bibo-Joshi marked this pull request as ready for review May 13, 2022 22:51
@Bibo-Joshi Bibo-Joshi requested review from Poolitzer and harshil21 and removed request for Poolitzer May 13, 2022 22:52
Copy link
Member

@harshil21 harshil21 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice. Now we can use this in examples to enforce the correct version.

@Bibo-Joshi
Copy link
Member Author

preview of the updated docs:

image

image

@Bibo-Joshi
Copy link
Member Author

@Poolitzer are you okay with the renaming tg.bot_api_version -> tg.__bot_api_version__ as well?

Copy link
Member

@Bibo-Joshi Havent looked into the specifics of the implementation but I like the idea jup

@Bibo-Joshi Bibo-Joshi merged commit 5e92401 into master May 14, 2022
@Bibo-Joshi Bibo-Joshi deleted the version-info branch May 14, 2022 13:50
@github-actions github-actions bot locked and limited conversation to collaborators May 21, 2022
@Bibo-Joshi Bibo-Joshi added 🔌 enhancement pr description: enhancement and removed enhancement labels Nov 3, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

🔌 enhancement pr description: enhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants