Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Deprecate python-telegram-bot-raw #4270

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 5 commits into from
Jun 1, 2024
Merged

Deprecate python-telegram-bot-raw #4270

merged 5 commits into from
Jun 1, 2024

Conversation

Bibo-Joshi
Copy link
Member

@Bibo-Joshi Bibo-Joshi commented May 22, 2024

Adresses first part of #4129

Manually tested the warnings. Automated testing doesn't seem to be very straight forward at first glance, so I'm willing to skip it …

  • Update the link to the channel post before merging

Draft for a channel post:

Hi all.

We have decided to deprecate the python-telegram-bot-raw library and no longer provide updates for it.

In Februrary 2021, we introduced the library python-telegram-bot-raw that provides the telegram module without telegram.ext. Back then, we felt this to be right thing to do since the rather new v13 introduced additional required dependencies for the JobQueue.

Since then v20 has seen the light of day however. With v20, python-telegram-bot only has a single required dependency, httpx for the networking backend. All other dependencies for further functionality of telegram.ext are purely optional. Moreover, according to the numbers that we are aware of ([1], [2], [3], [4]), the python-telegram-bot-raw library never gained a considerable user base compared to python-telegram-bot.

Since maintaining python-telegram-bot-raw still consumes some resources and is currently also hindering the modernization of PTBs packaging setup, we have therefore made this decision. The next release of PTB will add a corresponding warning to python-telegram-bot-raw. This will be the last update for python-telegram-bot-raw.

What does this mean to you?

Almost nothing. All you have to do is remove that trailing -raw from your requirements file. That's it. No code changes are necessary, since python-telegram-bot-raw is a subset of python-telegram-bot. No additional dependencies will be installed.

Thank you for your understanding and your continued usage of the python-telegram-bot library.
Your PTB Developer Team

@Bibo-Joshi Bibo-Joshi added 📋 do-not-merge-yet work status: do-not-merge-yet 🛠 refactor change type: refactor labels May 22, 2024
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Hey! Looks like you edited README.rst or README_RAW.rst. I'm just a friendly reminder to apply relevant changes to both of those files :)

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.

channel post seems good. edit: I edited OP to fix a typo

@aelkheir
Copy link
Member

aelkheir commented May 24, 2024

Some further typos for the channel post :)

since then v20 has seen the day of light.

light of day

Since mainaining python-telegram-bot-raw

maintaining

Thank you for your understand and

understanding

Copy link
Member

@Poolitzer Poolitzer left a comment

Choose a reason for hiding this comment

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

LGTM

@Bibo-Joshi
Copy link
Member Author

I built the wheels and double checked the installation on linux & windows. Merging.

@Bibo-Joshi Bibo-Joshi removed the 📋 do-not-merge-yet work status: do-not-merge-yet label Jun 1, 2024
@Bibo-Joshi Bibo-Joshi merged commit 57298aa into master Jun 1, 2024
21 of 25 checks passed
@Bibo-Joshi Bibo-Joshi deleted the deprecate-ptb-raw branch June 1, 2024 19:19
@github-actions github-actions bot locked and limited conversation to collaborators Jun 9, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
🛠 refactor change type: refactor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants