Skip to content

Commit

Permalink
fix: minor touchups based on stricter mypy settings
Browse files Browse the repository at this point in the history
  • Loading branch information
henryiii committed Mar 22, 2022
1 parent 358362d commit 38f0925
Show file tree
Hide file tree
Showing 4 changed files with 13 additions and 9 deletions.
11 changes: 5 additions & 6 deletions .pre-commit-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,8 @@ repos:
rev: v1.2.5
hooks:
- id: pycln
args: [--config=pyproject.toml]
args: [--all]
stages: [manual]

- repo: https://github.com/PyCQA/isort
rev: 5.10.1
Expand All @@ -53,7 +54,7 @@ repos:
- id: mypy
name: mypy 3.6 on cibuildwheel/
exclude: ^(bin|cibuildwheel/resources|docs)/.*py$
args: ["--python-version=3.6", "--show-error-codes"]
args: ["--python-version=3.6"]
additional_dependencies: &mypy-dependencies
- nox
- packaging>=21.0
Expand All @@ -71,11 +72,11 @@ repos:
- id: mypy
name: mypy 3.7+ on bin/
files: ^((bin|docs)/.*py)$
args: ["--python-version=3.7", "--show-error-codes"]
args: ["--python-version=3.7"]
additional_dependencies: *mypy-dependencies
- id: mypy
name: mypy 3.10
args: ["--python-version=3.10", "--show-error-codes"]
args: ["--python-version=3.10"]
additional_dependencies: *mypy-dependencies

- repo: https://github.com/asottile/yesqa
Expand All @@ -97,8 +98,6 @@ repos:
hooks:
- id: python-check-blanket-noqa
stages: [manual]
- id: python-check-blanket-type-ignore
stages: [manual]
- id: python-no-log-warn
- id: python-no-eval
- id: python-use-type-annotations
Expand Down
4 changes: 2 additions & 2 deletions cibuildwheel/options.py
Original file line number Diff line number Diff line change
Expand Up @@ -96,7 +96,7 @@ def architectures(self) -> Set[Architecture]:
return self.globals.architectures


Setting = Union[Dict[str, str], List[str], str]
Setting = Union[Dict[str, str], List[str], str, int]


class Override(NamedTuple):
Expand Down Expand Up @@ -496,7 +496,7 @@ def build_options(self, identifier: Optional[str]) -> BuildOptions:

config_value = self.reader.get(f"musllinux-{build_platform}-image")

if config_value is None:
if not config_value:
image = pinned_images["musllinux_1_1"]
elif config_value in pinned_images:
image = pinned_images[config_value]
Expand Down
5 changes: 5 additions & 0 deletions pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -38,6 +38,8 @@ files = [
"noxfile.py",
]
warn_unused_configs = true
show_error_codes = true

warn_redundant_casts = true
no_implicit_reexport = true
strict_equality = true
Expand All @@ -49,6 +51,9 @@ disallow_any_generics = true
warn_return_any = true
no_implicit_optional = true

enable_error_code = ["ignore-without-code", "redundant-expr", "truthy-bool"]
warn_unreachable = true

[[tool.mypy.overrides]]
module = "cibuildwheel.*"
disallow_untyped_defs = true
Expand Down
2 changes: 1 addition & 1 deletion unit_test/wheel_print_test.py
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ def test_no_printout_on_error(tmp_path, capsys):
tmp_path.joinpath("example.1").touch()
raise RuntimeError()

captured = capsys.readouterr()
captured = capsys.readouterr() # type: ignore[unreachable]
assert captured.err == ""

assert "example.0" not in captured.out
Expand Down

0 comments on commit 38f0925

Please sign in to comment.