Skip to content

Commit 841f512

Browse files
Remove pragma: no cover comments and type: ignore comments from setup_cfg/deps.py
Agent-Logs-Url: https://github.com/usethis-python/usethis-python/sessions/dddc5605-30c3-4d38-9d88-5e6b7fc9b894 Co-authored-by: nathanjmcdougall <18602289+nathanjmcdougall@users.noreply.github.com>
1 parent bfb2905 commit 841f512

1 file changed

Lines changed: 5 additions & 9 deletions

File tree

src/usethis/_file/setup_cfg/deps.py

Lines changed: 5 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -25,10 +25,8 @@ def get_setup_cfg_project_deps() -> list[Dependency]:
2525
if "install_requires" not in cfg["options"]:
2626
return []
2727

28-
raw_value: str | None = cfg["options"]["install_requires"].value # type: ignore[union-attr]
29-
if raw_value is None: # pragma: no cover
30-
return []
31-
return _parse_deps_string(raw_value)
28+
raw_value = cfg["options"]["install_requires"].value
29+
return _parse_deps_string(raw_value or "")
3230

3331

3432
def get_setup_cfg_dep_groups() -> dict[str, list[Dependency]]:
@@ -47,11 +45,9 @@ def get_setup_cfg_dep_groups() -> dict[str, list[Dependency]]:
4745
return {}
4846

4947
result: dict[str, list[Dependency]] = {}
50-
for extra_name in cfg["options.extras_require"].options(): # type: ignore[union-attr]
51-
raw_value: str | None = cfg["options.extras_require"][extra_name].value # type: ignore[union-attr]
52-
if raw_value is None: # pragma: no cover
53-
continue
54-
deps = _parse_deps_string(raw_value)
48+
for extra_name in cfg["options.extras_require"].options():
49+
raw_value = cfg["options.extras_require"][extra_name].value
50+
deps = _parse_deps_string(raw_value or "")
5551
if deps:
5652
result[extra_name] = deps
5753

0 commit comments

Comments
 (0)