🐴 simplified sys.platform checks

Author: jorenham

src/numpy-stubs/@test/static/accept/testing.pyi

@@ -165,8 +165,7 @@ assert_type(np.testing.break_cycles(), None)
 
 # NOTE: Platform-dependent behavior cannot be tested on pyright
 # ruff: noqa: ERA001
-
-# if sys.platform == "win32" or sys.platform == "cygwin":
+# if sys.platform == "win32":
 #     assert_type(np.testing.memusage(), int)
 # elif sys.platform == "linux":
 #     assert_type(np.testing.memusage(), int | None)

src/numpy-stubs/__init__.pyi

@@ -596,7 +596,7 @@ __all__: list[str] = [  # noqa: RUF022
     "__version__",
 ]  # fmt: skip
 
-if sys.platform == "win32" or sys.platform == "cygwin":
+if sys.platform == "win32":
     __all__ += ["complex192", "float96"]
 else:
     __all__ += ["complex256", "float128"]
@@ -4704,7 +4704,7 @@ class longdouble(_FloatMixin[L[12, 16]], floating):
     @override
     def tolist(self, /) -> Self: ...  # pyright: ignore[reportIncompatibleMethodOverride]
 
-if sys.platform == "win32" or sys.platform == "cygwin":
+if sys.platform == "win32":
     float96 = longdouble
 else:
     float128 = longdouble
@@ -4894,7 +4894,7 @@ class clongdouble(complexfloating):
     #
     def __complex__(self, /) -> complex: ...
 
-if sys.platform == "win32" or sys.platform == "cygwin":
+if sys.platform == "win32":
     complex192 = clongdouble
 else:
     complex256 = clongdouble

src/numpy-stubs/_core/__init__.pyi

@@ -641,7 +641,7 @@ __all__: list[str] = [
     "zeros_like",
 ]
 
-if sys.platform == "win32" or sys.platform == "cygwin":
+if sys.platform == "win32":
     from .numerictypes import complex192, float96
 
     __all__ += ["complex192", "float96"]

src/numpy-stubs/_core/numeric.pyi

@@ -583,7 +583,7 @@ __all__: list[str] = [
     "zeros_like",
 ]
 
-if sys.platform == "win32" or sys.platform == "cygwin":
+if sys.platform == "win32":
     from .numerictypes import complex192, float96
 
     __all__ += ["complex192", "float96"]

src/numpy-stubs/_core/numerictypes.pyi

@@ -131,7 +131,7 @@ __all__: list[str] = [
     "void",
 ]
 
-if sys.platform == "win32" or sys.platform == "cygwin":
+if sys.platform == "win32":
     from numpy import complex192, float96
 
     __all__ += ["complex192", "float96"]

src/numpy-stubs/core/numeric.pyi

@@ -292,7 +292,8 @@ from numpy._core.numeric import (
     zeros_like,
 )
 
-__all__ = [
+# pyright needs the `list[str]` annotation here
+__all__: list[str] = [
     "False_",
     "ScalarType",
     "True_",
@@ -585,7 +586,7 @@ __all__ = [
     "zeros_like",
 ]
 
-if sys.platform == "win32" or sys.platform == "cygwin":
+if sys.platform == "win32":
     from numpy._core.numeric import complex192, float96
 
     __all__ += ["complex192", "float96"]

src/numpy-stubs/core/numerictypes.pyi

@@ -65,7 +65,8 @@ from numpy._core.numerictypes import (
     void,
 )
 
-__all__ = [  # noqa: RUF022
+# pyright needs the `list[str]` annotation here
+__all__: list[str] = [  # noqa: RUF022
     "ScalarType",
     "typecodes",
     "isdtype",
@@ -130,7 +131,7 @@ __all__ = [  # noqa: RUF022
     "timedelta64",
 ]
 
-if sys.platform == "win32" or sys.platform == "cygwin":
+if sys.platform == "win32":
     from numpy._core.numerictypes import complex192, float96
 
     __all__ += ["complex192", "float96"]

src/numpy-stubs/matlib.pyi

@@ -600,7 +600,7 @@ __all__ += [  # noqa: RUF022
     "__version__",
 ]  # fmt: skip
 
-if sys.platform == "win32" or sys.platform == "cygwin":
+if sys.platform == "win32":
     from numpy import complex192, float96
 
     __all__ += ["complex192", "float96"]

src/numpy-stubs/testing/_private/utils.pyi

@@ -160,12 +160,11 @@ class suppress_warnings:
         self, /, category: type[Warning] = ..., message: str = "", module: types.ModuleType | None = None
     ) -> _WarnLog: ...
 
-# Contrary to runtime we can't do `os.name` checks while type checking,
-# only `sys.platform` checks
-if sys.platform != "win32" and sys.platform != "cygwin" and sys.platform != "linux":
+# At runtime `os.name` is used, but static type checkers only support a small subset of `sys.platform` checks.
+if sys.platform != "win32" and sys.platform != "linux":
     def memusage() -> NoReturn: ...
 
-elif sys.platform == "win32" or sys.platform == "cygwin":
+elif sys.platform == "win32":
     def memusage(processName: str = "python", instance: int = 0) -> int: ...
 
 else: