fix: open setup.py as utf-8 in get_requires_python_str (#977)
* open setup.py as utf-8 in get_requires_python_str fix #976 * update test for UTF-8 setup.py Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Co-authored-by: mayeut <mayeut@users.noreply.github.com>
This commit is contained in:
co-authored by
pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
mayeut
parent
d6bede539d
commit
e9ef3e9750
@@ -70,7 +70,7 @@ def get_requires_python_str(package_dir: Path) -> Optional[str]:
|
|||||||
pass
|
pass
|
||||||
|
|
||||||
try:
|
try:
|
||||||
with (package_dir / "setup.py").open() as f2:
|
with (package_dir / "setup.py").open(encoding="utf8") as f2:
|
||||||
return setup_py_python_requires(f2.read())
|
return setup_py_python_requires(f2.read())
|
||||||
except FileNotFoundError:
|
except FileNotFoundError:
|
||||||
pass
|
pass
|
||||||
|
|||||||
@@ -25,7 +25,7 @@ def test_read_setup_py_simple(tmp_path):
|
|||||||
|
|
||||||
|
|
||||||
def test_read_setup_py_full(tmp_path):
|
def test_read_setup_py_full(tmp_path):
|
||||||
with open(tmp_path / "setup.py", "w") as f:
|
with open(tmp_path / "setup.py", "w", encoding="utf8") as f:
|
||||||
f.write(
|
f.write(
|
||||||
dedent(
|
dedent(
|
||||||
"""
|
"""
|
||||||
@@ -35,6 +35,7 @@ def test_read_setup_py_full(tmp_path):
|
|||||||
|
|
||||||
setuptools.setup(
|
setuptools.setup(
|
||||||
name = "hello",
|
name = "hello",
|
||||||
|
description = "≥“”ü",
|
||||||
other = 23,
|
other = 23,
|
||||||
example = ["item", "other"],
|
example = ["item", "other"],
|
||||||
python_requires = "1.24",
|
python_requires = "1.24",
|
||||||
@@ -43,7 +44,9 @@ def test_read_setup_py_full(tmp_path):
|
|||||||
)
|
)
|
||||||
)
|
)
|
||||||
|
|
||||||
assert setup_py_python_requires(tmp_path.joinpath("setup.py").read_text()) == "1.24"
|
assert (
|
||||||
|
setup_py_python_requires(tmp_path.joinpath("setup.py").read_text(encoding="utf8")) == "1.24"
|
||||||
|
)
|
||||||
assert get_requires_python_str(tmp_path) == "1.24"
|
assert get_requires_python_str(tmp_path) == "1.24"
|
||||||
|
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user