mirror/python-telegram-bot
1
0
Fork 0
mirror of https://github.com/python-telegram-bot/python-telegram-bot.git synced 2026-06-19 07:35:19 +00:00
Code Issues Packages Projects Releases Wiki Activity

Bump pre-commit Hooks to Latest Versions (#4643)

Browse Source 
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Hinrich Mahler <22366557+Bibo-Joshi@users.noreply.github.com>
This commit is contained in:
pre-commit-ci[bot]
2025-01-07 18:11:35 +01:00
committed by GitHub
parent e4b0f8cb64
commit 16605c54d7
15 changed files with 36 additions and 38 deletions
Download Patch File Download Diff File Expand all files Collapse all files
.pre-commit-config.yaml
+6 -6
View File
@@ -7,7 +7,7 @@ ci:
repos:
- repo: https://github.com/astral-sh/ruff-pre-commit
rev: 'v0.5.6'
rev: 'v0.8.6'
hooks:
- id: ruff
name: ruff
@@ -18,18 +18,18 @@ repos:
- cachetools>=5.3.3,<5.5.0
- aiolimiter~=1.1,<1.3
- repo: https://github.com/psf/black-pre-commit-mirror
rev: 24.4.2
rev: 24.10.0
hooks:
- id: black
args:
- --diff
- --check
- repo: https://github.com/PyCQA/flake8
rev: 7.1.0
rev: 7.1.1
hooks:
- id: flake8
- repo: https://github.com/PyCQA/pylint
rev: v3.3.2
rev: v3.3.3
hooks:
- id: pylint
files: ^(?!(tests|docs)).*\.py$
@@ -41,7 +41,7 @@ repos:
- aiolimiter~=1.1,<1.3
- . # this basically does `pip install -e .`
- repo: https://github.com/pre-commit/mirrors-mypy
rev: v1.10.1
rev: v1.14.1
hooks:
- id: mypy
name: mypy-ptb
@@ -68,7 +68,7 @@ repos:
- cachetools>=5.3.3,<5.5.0
- . # this basically does `pip install -e .`
- repo: https://github.com/asottile/pyupgrade
rev: v3.16.0
rev: v3.19.1
hooks:
- id: pyupgrade
args:
docs/auxil/admonition_inserter.py
+5 -5
View File
@@ -435,12 +435,12 @@ class AdmonitionInserter:
return admonition_for_class
@staticmethod
def _generate_class_name_for_link(cls: type) -> str:
def _generate_class_name_for_link(cls_: type) -> str:
"""Generates class name that can be used in a ReST link."""
# Check for potential presence of ".ext.", we will need to keep it.
ext = ".ext" if ".ext." in str(cls) else ""
return f"telegram{ext}.{cls.__name__}"
ext = ".ext" if ".ext." in str(cls_) else ""
return f"telegram{ext}.{cls_.__name__}"
def _generate_link_to_method(self, method_name: str, cls: type) -> str:
"""Generates a ReST link to a method of a telegram class."""
@@ -448,11 +448,11 @@ class AdmonitionInserter:
return f":meth:`{self._generate_class_name_for_link(cls)}.{method_name}`"
@staticmethod
def _iter_subclasses(cls: type) -> Iterator:
def _iter_subclasses(cls_: type) -> Iterator:
return (
# exclude private classes
c
for c in cls.__subclasses__()
for c in cls_.__subclasses__()
if not str(c).split(".")[-1].startswith("_")
)
pyproject.toml
+1 -1
View File
@@ -10,7 +10,7 @@ description = "We have made you a wrapper you can't refuse"
readme = "README.rst"
requires-python = ">=3.9"
license = "LGPL-3.0-only"
license-files = { paths = ["LICENSE", "LICENSE.dual", "LICENSE.lesser"] }
license-files = ["LICENSE", "LICENSE.dual", "LICENSE.lesser"]
authors = [
{ name = "Leandro Toledo", email = "devs@python-telegram-bot.org" }
]
telegram/_bot.py
+1 -1
View File
@@ -4386,7 +4386,7 @@ class Bot(TelegramObject, contextlib.AbstractAsyncContextManager["Bot"]):
self,
offset: Optional[int] = None,
limit: Optional[int] = None,
timeout: Optional[int] = None, # noqa: ASYNC109
timeout: Optional[int] = None,
allowed_updates: Optional[Sequence[str]] = None,
*,
read_timeout: ODVInput[float] = DEFAULT_NONE,
telegram/ext/_aioratelimiter.py
+1 -1
View File
@@ -247,7 +247,7 @@ class AIORateLimiter(BaseRateLimiter[int]):
# In case user passes integer chat id as string
with contextlib.suppress(ValueError, TypeError):
chat_id = int(chat_id)
chat_id = int(chat_id) # type: ignore[arg-type]
if (isinstance(chat_id, int) and chat_id < 0) or isinstance(chat_id, str):
# string chat_id only works for channels and supergroups
telegram/ext/_callbackcontext.py
+1 -1
View File
@@ -145,7 +145,7 @@ class CallbackContext(Generic[BT, UD, CD, BD]):
@property
def application(self) -> "Application[BT, ST, UD, CD, BD, Any]":
""":class:`telegram.ext.Application`: The application associated with this context."""
return self._application
return self._application # type: ignore[return-value]
@property
def bot_data(self) -> BD:
telegram/ext/_extbot.py
+1 -1
View File
@@ -638,7 +638,7 @@ class ExtBot(Bot, Generic[RLARGS]):
self,
offset: Optional[int] = None,
limit: Optional[int] = None,
timeout: Optional[int] = None, # noqa: ASYNC109
timeout: Optional[int] = None,
allowed_updates: Optional[Sequence[str]] = None,
*,
read_timeout: ODVInput[float] = DEFAULT_NONE,
telegram/ext/_updater.py
+2 -2
View File
@@ -205,7 +205,7 @@ class Updater(contextlib.AbstractAsyncContextManager["Updater"]):
async def start_polling(
self,
poll_interval: float = 0.0,
timeout: int = 10, # noqa: ASYNC109
timeout: int = 10,
bootstrap_retries: int = -1,
read_timeout: ODVInput[float] = DEFAULT_NONE,
write_timeout: ODVInput[float] = DEFAULT_NONE,
@@ -341,7 +341,7 @@ class Updater(contextlib.AbstractAsyncContextManager["Updater"]):
async def _start_polling(
self,
poll_interval: float,
timeout: int, # noqa: ASYNC109
timeout: int,
read_timeout: ODVInput[float],
write_timeout: ODVInput[float],
connect_timeout: ODVInput[float],
tests/conftest.py
+1 -1
View File
@@ -311,7 +311,7 @@ def false_update(request):
scope="session",
params=[pytz.timezone, zoneinfo.ZoneInfo] if TEST_WITH_OPT_DEPS else [zoneinfo.ZoneInfo],
)
def _tz_implementation(request): # noqa: PT005
def _tz_implementation(request):
# This fixture is used to parametrize the timezone fixture
# This is similar to what @pyttest.mark.parametrize does but for fixtures
# However, this is needed only internally for the `tzinfo` fixture, so we keep it private
tests/ext/test_chatboosthandler.py
+1 -1
View File
@@ -132,7 +132,7 @@ class TestChatBoostHandler:
)
@pytest.mark.parametrize(
argnames=["allowed_types", "cb", "expected"],
argnames=("allowed_types", "cb", "expected"),
argvalues=[
(ChatBoostHandler.CHAT_BOOST, "cb_chat_boost_updated", (True, False)),
(ChatBoostHandler.REMOVED_CHAT_BOOST, "cb_chat_boost_removed", (False, True)),
tests/ext/test_chatmemberhandler.py
+2 -2
View File
@@ -115,7 +115,7 @@ class TestChatMemberHandler:
)
@pytest.mark.parametrize(
argnames=["allowed_types", "expected"],
argnames=("allowed_types", "expected"),
argvalues=[
(ChatMemberHandler.MY_CHAT_MEMBER, (True, False)),
(ChatMemberHandler.CHAT_MEMBER, (False, True)),
@@ -145,7 +145,7 @@ class TestChatMemberHandler:
assert self.test_flag == result_2
@pytest.mark.parametrize(
argnames=["allowed_types", "chat_id", "expected"],
argnames=("allowed_types", "chat_id", "expected"),
argvalues=[
(ChatMemberHandler.MY_CHAT_MEMBER, None, (True, False)),
(ChatMemberHandler.CHAT_MEMBER, None, (False, True)),
tests/ext/test_messagereactionhandler.py
+8 -8
View File
@@ -173,7 +173,7 @@ class TestMessageReactionHandler:
assert self.test_flag
@pytest.mark.parametrize(
argnames=["allowed_types", "expected"],
argnames=("allowed_types", "expected"),
argvalues=[
(MessageReactionHandler.MESSAGE_REACTION_UPDATED, (True, False)),
(MessageReactionHandler.MESSAGE_REACTION_COUNT_UPDATED, (False, True)),
@@ -201,7 +201,7 @@ class TestMessageReactionHandler:
assert self.test_flag == result_2
@pytest.mark.parametrize(
argnames=["allowed_types", "kwargs"],
argnames=("allowed_types", "kwargs"),
argvalues=[
(MessageReactionHandler.MESSAGE_REACTION_COUNT_UPDATED, {"user_username": "user"}),
(MessageReactionHandler.MESSAGE_REACTION, {"user_id": 123}),
@@ -215,7 +215,7 @@ class TestMessageReactionHandler:
MessageReactionHandler(self.callback, message_reaction_types=allowed_types, **kwargs)
@pytest.mark.parametrize(
argnames=["chat_id", "expected"],
argnames=("chat_id", "expected"),
argvalues=[(1, True), ([1], True), (2, False), ([2], False)],
)
async def test_with_chat_ids(
@@ -226,8 +226,8 @@ class TestMessageReactionHandler:
assert handler.check_update(message_reaction_count_update) == expected
@pytest.mark.parametrize(
argnames=["chat_username"],
argvalues=[("group_a",), ("@group_a",), (["group_a"],), (["@group_a"],)],
argnames="chat_username",
argvalues=["group_a", "@group_a", ["group_a"], ["@group_a"]],
ids=["group_a", "@group_a", "['group_a']", "['@group_a']"],
)
async def test_with_chat_usernames(
@@ -247,7 +247,7 @@ class TestMessageReactionHandler:
message_reaction_count_update.message_reaction_count.chat.username = None
@pytest.mark.parametrize(
argnames=["user_id", "expected"],
argnames=("user_id", "expected"),
argvalues=[(1, True), ([1], True), (2, False), ([2], False)],
)
async def test_with_user_ids(
@@ -262,8 +262,8 @@ class TestMessageReactionHandler:
assert not handler.check_update(message_reaction_count_update)
@pytest.mark.parametrize(
argnames=["user_username"],
argvalues=[("user_a",), ("@user_a",), (["user_a"],), (["@user_a"],)],
argnames="user_username",
argvalues=["user_a", "@user_a", ["user_a"], ["@user_a"]],
ids=["user_a", "@user_a", "['user_a']", "['@user_a']"],
)
async def test_with_user_usernames(
tests/test_bot.py
+1 -1
View File
@@ -191,7 +191,7 @@ def bot_methods(ext_bot=True, include_camel_case=False, include_do_api_request=F
ids.append(f"{cls.__name__}.{name}")
return pytest.mark.parametrize(
argnames="bot_class, bot_method_name,bot_method", argvalues=arg_values, ids=ids
argnames=("bot_class", "bot_method_name", "bot_method"), argvalues=arg_values, ids=ids
)
tests/test_chat.py
+3 -3
View File
@@ -286,7 +286,7 @@ class TestChatWithoutRequest(ChatTestBase):
async def make_assertion(*_, **kwargs):
chat_id = kwargs["chat_id"] == chat.id
user_id = kwargs["user_id"] == 42
o_i_b = kwargs.get("only_if_banned", None) == only_if_banned
o_i_b = kwargs.get("only_if_banned") == only_if_banned
return chat_id and user_id and o_i_b
assert check_shortcut_signature(Chat.unban_member, Bot.unban_chat_member, ["chat_id"], [])
@@ -333,7 +333,7 @@ class TestChatWithoutRequest(ChatTestBase):
async def make_assertion(*_, **kwargs):
chat_id = kwargs["chat_id"] == chat.id
user_id = kwargs["user_id"] == 42
o_i_b = kwargs.get("is_anonymous", None) == is_anonymous
o_i_b = kwargs.get("is_anonymous") == is_anonymous
return chat_id and user_id and o_i_b
assert check_shortcut_signature(
@@ -353,7 +353,7 @@ class TestChatWithoutRequest(ChatTestBase):
async def make_assertion(*_, **kwargs):
chat_id = kwargs["chat_id"] == chat.id
user_id = kwargs["user_id"] == 42
o_i_b = kwargs.get("permissions", None) == permissions
o_i_b = kwargs.get("permissions") == permissions
return chat_id and user_id and o_i_b
assert check_shortcut_signature(
tests/test_chatmember.py
+2 -4
View File
@@ -153,10 +153,8 @@ def make_json_dict(instance: ChatMember, include_optional_args: bool = False) ->
# If we want to test all args (for de_json)
# or if the param is optional but for backwards compatability
elif (
param.default is not inspect.Parameter.empty
and include_optional_args
or param.name in ["can_delete_stories", "can_post_stories", "can_edit_stories"]
):
param.default is not inspect.Parameter.empty and include_optional_args
) or param.name in ["can_delete_stories", "can_post_stories", "can_edit_stories"]:
json_dict[param.name] = val
return json_dict