Skip to content
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

Make sure PP can read files that dont have bot_data #1760

Merged
merged 2 commits into from
Feb 8, 2020

Conversation

Bibo-Joshi
Copy link
Member

@Bibo-Joshi Bibo-Joshi commented Feb 8, 2020

This assures that a pickle file that doesn't contain bot_data is read correctly. Tests are added as well.
Not necessary for DictPersistence because it doesn't store anything.

closes #1761

telegram/ext/picklepersistence.py Outdated Show resolved Hide resolved
telegram/ext/picklepersistence.py Outdated Show resolved Hide resolved
tests/test_persistence.py Show resolved Hide resolved
tests/test_persistence.py Show resolved Hide resolved
Copy link
Member

@tsnoam tsnoam left a comment

Choose a reason for hiding this comment

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

We can merge if ci succeeds

@tsnoam tsnoam merged commit a447760 into master Feb 8, 2020
@tsnoam tsnoam deleted the fix-missing-bot-data branch February 8, 2020 17:24
@github-actions github-actions bot locked and limited conversation to collaborators Aug 19, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] v12.4 breaks PicklePersistence
2 participants