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

API 6.1 for V13 #3117

Merged
merged 10 commits into from Jun 28, 2022
Merged

API 6.1 for V13 #3117

merged 10 commits into from Jun 28, 2022

Conversation

Poolitzer
Copy link
Member

Same PR as #3112, just with V13

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 :)

@Poolitzer Poolitzer changed the title Fix: Change constant variable naming API 6.1 for V13 Jun 24, 2022
setup.cfg Show resolved Hide resolved
telegram/bot.py Show resolved Hide resolved
telegram/bot.py Outdated Show resolved Hide resolved
telegram/bot.py Outdated Show resolved Hide resolved
telegram/constants.py Show resolved Hide resolved
telegram/ext/filters.py Outdated Show resolved Hide resolved
telegram/ext/filters.py Outdated Show resolved Hide resolved
telegram/ext/filters.py Outdated Show resolved Hide resolved
telegram/ext/filters.py Outdated Show resolved Hide resolved
telegram/ext/filters.py Outdated Show resolved Hide resolved
tests/test_invoice.py Outdated Show resolved Hide resolved
Poolitzer and others added 4 commits June 26, 2022 14:57
Co-authored-by: Harshil <37377066+harshil21@users.noreply.github.com>
Co-authored-by: Harshil <37377066+harshil21@users.noreply.github.com>
telegram/constants.py Outdated Show resolved Hide resolved
Co-authored-by: Harshil <37377066+harshil21@users.noreply.github.com>
@Bibo-Joshi
Copy link
Member

Why is pre-commit failing? I thought we had managed that in #3027?

@Poolitzer
Copy link
Member Author

Is it the same error or was it a different one?

@Bibo-Joshi
Copy link
Member

On this PR it's

telegram/__main__.py:44: error: unused 'type: ignore' comment
Found 1 error in 1 file (checked 155 source files)

on #3027 it was

telegram/__main__.py:44: error: Module has no attribute "__version__"  [attr-defined]
Found 9 errors in 2 files (checked 155 source files)

@Poolitzer
Copy link
Member Author

Yeah I don't know, it works perfectly for pycharm, and it definitely fails locally when I remove the comment.

Oh hey look at that it works on CI without it.

@Bibo-Joshi Bibo-Joshi merged commit debe86a into v13.x Jun 28, 2022
@Bibo-Joshi Bibo-Joshi deleted the v13_api_6.1 branch June 28, 2022 17:34
@github-actions github-actions bot locked and limited conversation to collaborators Jul 5, 2022
@harshil21 harshil21 added this to the 13.13 milestone Aug 29, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants