chore: avoid importing TYPE_CHECKING (#2864)
Assisted-by: OpenCode:glm-5.1 Signed-off-by: Henry Schreiner <henryfs@princeton.edu>
This commit is contained in:
@@ -6,7 +6,6 @@ import sys
|
|||||||
import tarfile
|
import tarfile
|
||||||
|
|
||||||
TYPE_CHECKING = False
|
TYPE_CHECKING = False
|
||||||
|
|
||||||
if TYPE_CHECKING:
|
if TYPE_CHECKING:
|
||||||
from pathlib import Path
|
from pathlib import Path
|
||||||
|
|
||||||
|
|||||||
@@ -11,12 +11,13 @@ import textwrap
|
|||||||
import time
|
import time
|
||||||
from collections.abc import Generator
|
from collections.abc import Generator
|
||||||
from pathlib import Path
|
from pathlib import Path
|
||||||
from typing import IO, TYPE_CHECKING, AnyStr, Final, Literal
|
from typing import IO, AnyStr, Final, Literal
|
||||||
|
|
||||||
import humanize
|
import humanize
|
||||||
|
|
||||||
from cibuildwheel.ci import CIProvider, detect_ci_provider, filter_ansi_codes
|
from cibuildwheel.ci import CIProvider, detect_ci_provider, filter_ansi_codes
|
||||||
|
|
||||||
|
TYPE_CHECKING = False
|
||||||
if TYPE_CHECKING:
|
if TYPE_CHECKING:
|
||||||
from cibuildwheel.options import Options
|
from cibuildwheel.options import Options
|
||||||
|
|
||||||
|
|||||||
@@ -7,7 +7,7 @@ import textwrap
|
|||||||
from collections import OrderedDict
|
from collections import OrderedDict
|
||||||
from collections.abc import Iterable, Iterator, Sequence, Set
|
from collections.abc import Iterable, Iterator, Sequence, Set
|
||||||
from pathlib import Path, PurePath, PurePosixPath
|
from pathlib import Path, PurePath, PurePosixPath
|
||||||
from typing import TYPE_CHECKING, assert_never
|
from typing import assert_never
|
||||||
|
|
||||||
from cibuildwheel import errors
|
from cibuildwheel import errors
|
||||||
from cibuildwheel.architecture import Architecture
|
from cibuildwheel.architecture import Architecture
|
||||||
@@ -22,6 +22,7 @@ from cibuildwheel.util.file import copy_test_sources
|
|||||||
from cibuildwheel.util.helpers import prepare_command, unwrap
|
from cibuildwheel.util.helpers import prepare_command, unwrap
|
||||||
from cibuildwheel.util.packaging import find_compatible_wheel
|
from cibuildwheel.util.packaging import find_compatible_wheel
|
||||||
|
|
||||||
|
TYPE_CHECKING = False
|
||||||
if TYPE_CHECKING:
|
if TYPE_CHECKING:
|
||||||
from cibuildwheel.typing import PathOrStr
|
from cibuildwheel.typing import PathOrStr
|
||||||
|
|
||||||
|
|||||||
@@ -231,6 +231,7 @@ flake8-annotations.mypy-init-return = true
|
|||||||
"typing.Iterator".msg = "Use collections.abc.Iterator instead."
|
"typing.Iterator".msg = "Use collections.abc.Iterator instead."
|
||||||
"typing.Sequence".msg = "Use collections.abc.Sequence instead."
|
"typing.Sequence".msg = "Use collections.abc.Sequence instead."
|
||||||
"typing.Set".msg = "Use collections.abc.Set instead."
|
"typing.Set".msg = "Use collections.abc.Set instead."
|
||||||
|
"typing.TYPE_CHECKING".msg = "Use TYPE_CHECKING=False instead"
|
||||||
|
|
||||||
[tool.ruff.lint.per-file-ignores]
|
[tool.ruff.lint.per-file-ignores]
|
||||||
"unit_test/*" = ["PLC1901", "TID252"]
|
"unit_test/*" = ["PLC1901", "TID252"]
|
||||||
|
|||||||
@@ -2,7 +2,6 @@ import contextlib
|
|||||||
import sys
|
import sys
|
||||||
from collections.abc import Generator
|
from collections.abc import Generator
|
||||||
from pathlib import Path
|
from pathlib import Path
|
||||||
from typing import TYPE_CHECKING
|
|
||||||
|
|
||||||
import pytest
|
import pytest
|
||||||
import setuptools._distutils.util
|
import setuptools._distutils.util
|
||||||
@@ -11,6 +10,8 @@ from cibuildwheel.ci import CIProvider, detect_ci_provider
|
|||||||
from cibuildwheel.errors import FatalError
|
from cibuildwheel.errors import FatalError
|
||||||
from cibuildwheel.platforms.windows import PythonConfiguration, setup_setuptools_cross_compile
|
from cibuildwheel.platforms.windows import PythonConfiguration, setup_setuptools_cross_compile
|
||||||
|
|
||||||
|
TYPE_CHECKING = False
|
||||||
|
|
||||||
# monkeypatching os.name is too flaky. E.g. It works on my machine, but fails in pipeline
|
# monkeypatching os.name is too flaky. E.g. It works on my machine, but fails in pipeline
|
||||||
if not sys.platform.startswith("win") and not TYPE_CHECKING:
|
if not sys.platform.startswith("win") and not TYPE_CHECKING:
|
||||||
pytest.skip("Windows-only tests", allow_module_level=True)
|
pytest.skip("Windows-only tests", allow_module_level=True)
|
||||||
|
|||||||
@@ -3,7 +3,6 @@ import tomllib
|
|||||||
from collections.abc import Mapping
|
from collections.abc import Mapping
|
||||||
from fnmatch import fnmatch
|
from fnmatch import fnmatch
|
||||||
from pathlib import Path
|
from pathlib import Path
|
||||||
from typing import TYPE_CHECKING
|
|
||||||
|
|
||||||
import pytest
|
import pytest
|
||||||
|
|
||||||
@@ -15,6 +14,7 @@ from cibuildwheel.selector import BuildSelector, EnableGroup
|
|||||||
from cibuildwheel.util import resources
|
from cibuildwheel.util import resources
|
||||||
from cibuildwheel.util.packaging import DependencyConstraints
|
from cibuildwheel.util.packaging import DependencyConstraints
|
||||||
|
|
||||||
|
TYPE_CHECKING = False
|
||||||
if TYPE_CHECKING:
|
if TYPE_CHECKING:
|
||||||
from .conftest import ArgsInterceptor
|
from .conftest import ArgsInterceptor
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user