Skip to content

Commit fd8607d

Browse files
authored
Fix mypy error from urllib.Retry kwargs (#858)
* Silence mypy kwargs error * Require recent version of urllib3 * Add packages to --version * Show version for all requirements
1 parent ce87465 commit fd8607d

File tree

3 files changed

+11
-19
lines changed

3 files changed

+11
-19
lines changed

setup.cfg

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -40,6 +40,7 @@ install_requires=
4040
readme_renderer >= 21.0
4141
requests >= 2.20
4242
requests-toolbelt >= 0.8.0, != 0.9.0
43+
urllib3 >= 1.26.0
4344
tqdm >= 4.14
4445
importlib_metadata >= 3.6
4546
keyring >= 15.1

twine/cli.py

Lines changed: 8 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -14,23 +14,18 @@
1414
import argparse
1515
from typing import Any, List, Tuple
1616

17-
from importlib_metadata import entry_points
18-
from importlib_metadata import version
17+
import importlib_metadata
18+
from packaging import requirements
1919

2020
import twine
2121

2222
args = argparse.Namespace()
2323

2424

2525
def list_dependencies_and_versions() -> List[Tuple[str, str]]:
26-
deps = (
27-
"importlib_metadata",
28-
"pkginfo",
29-
"requests",
30-
"requests-toolbelt",
31-
"tqdm",
32-
)
33-
return [(dep, version(dep)) for dep in deps] # type: ignore[no-untyped-call] # python/importlib_metadata#288 # noqa: E501
26+
requires = importlib_metadata.requires("twine") # type: ignore[no-untyped-call] # python/importlib_metadata#288 # noqa: E501
27+
deps = [requirements.Requirement(r).name for r in requires]
28+
return [(dep, importlib_metadata.version(dep)) for dep in deps] # type: ignore[no-untyped-call] # python/importlib_metadata#288 # noqa: E501
3429

3530

3631
def dep_versions() -> str:
@@ -40,7 +35,9 @@ def dep_versions() -> str:
4035

4136

4237
def dispatch(argv: List[str]) -> Any:
43-
registered_commands = entry_points(group="twine.registered_commands")
38+
registered_commands = importlib_metadata.entry_points(
39+
group="twine.registered_commands"
40+
)
4441
parser = argparse.ArgumentParser(prog="twine")
4542
parser.add_argument(
4643
"--version",

twine/repository.py

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -77,19 +77,13 @@ def __init__(
7777

7878
@staticmethod
7979
def _make_adapter_with_retries() -> adapters.HTTPAdapter:
80-
retry_kwargs = dict(
80+
retry = urllib3.Retry(
81+
allowed_methods=["GET"],
8182
connect=5,
8283
total=10,
8384
status_forcelist=[500, 501, 502, 503],
8485
)
8586

86-
try:
87-
retry = urllib3.Retry(allowed_methods=["GET"], **retry_kwargs)
88-
except TypeError: # pragma: no cover
89-
# Avoiding DeprecationWarning starting in urllib3 1.26
90-
# Remove when that's the mininum version
91-
retry = urllib3.Retry(method_whitelist=["GET"], **retry_kwargs)
92-
9387
return adapters.HTTPAdapter(max_retries=retry)
9488

9589
@staticmethod

0 commit comments

Comments
 (0)