Skip to content

Commit

Permalink
Fix
Browse files Browse the repository at this point in the history
Signed-off-by: Bernát Gábor <bgabor8@bloomberg.net>
  • Loading branch information
gaborbernat committed Jun 27, 2023
1 parent 4113349 commit 67c5cb3
Show file tree
Hide file tree
Showing 11 changed files with 30 additions and 23 deletions.
2 changes: 1 addition & 1 deletion .pre-commit-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ repos:
rev: "0.12.1"
hooks:
- id: pyproject-fmt
additional_dependencies: ["tox>=4.6.1"]
additional_dependencies: ["tox>=4.6.3"]
- repo: https://github.com/pre-commit/mirrors-prettier
rev: "v3.0.0-alpha.9-for-vscode"
hooks:
Expand Down
2 changes: 1 addition & 1 deletion docs/index.rst
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ and `devpi <https://www.devpi.net>`_).
:target: https://pypistats.org/packages/tox
:alt: PyPI - Downloads
.. image:: https://img.shields.io/pypi/l/tox?style=flat-square
:target: https://opensource.org/licenses/MIT
:target: https://opensource.org/license/mit/
:alt: PyPI - License
.. image:: https://img.shields.io/github/issues/tox-dev/tox?style=flat-square
:target: https://github.com/tox-dev/tox/issues
Expand Down
6 changes: 4 additions & 2 deletions docs/tox_conf.py
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
from __future__ import annotations

from typing import TYPE_CHECKING, cast
from typing import TYPE_CHECKING, Any, ClassVar, cast

from docutils.nodes import Element, Node, Text, container, fully_normalize_name, literal, paragraph, reference, strong
from docutils.parsers.rst.directives import flag, unchanged, unchanged_required
Expand All @@ -11,6 +11,8 @@
from sphinx.util.logging import getLogger

if TYPE_CHECKING:
from typing import Final

from docutils.parsers.rst.states import RSTState, RSTStateMachine

LOGGER = getLogger(__name__)
Expand All @@ -19,7 +21,7 @@
class ToxConfig(SphinxDirective):
name = "conf"
has_content = True
option_spec = {
option_spec: Final[ClassVar[dict[str, Any]]] = {
"keys": unchanged_required,
"version_added": unchanged,
"version_changed": unchanged,
Expand Down
10 changes: 5 additions & 5 deletions pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -52,10 +52,10 @@ dependencies = [
"chardet>=5.1",
"colorama>=0.4.6",
"filelock>=3.12.2",
'importlib-metadata>=6.6; python_version < "3.8"',
'importlib-metadata>=6.7; python_version < "3.8"',
"packaging>=23.1",
"platformdirs>=3.5.3",
"pluggy>=1",
"platformdirs>=3.8",
"pluggy>=1.2",
"pyproject-api>=1.5.2",
'tomli>=2.0.1; python_version < "3.11"',
'typing-extensions>=4.6.3; python_version < "3.8"',
Expand All @@ -65,7 +65,7 @@ optional-dependencies.docs = [
"furo>=2023.5.20",
"sphinx>=7.0.1",
"sphinx-argparse-cli>=1.11.1",
"sphinx-autodoc-typehints!=1.23.4,>=1.23.2",
"sphinx-autodoc-typehints!=1.23.4,>=1.23.3",
"sphinx-copybutton>=0.5.2",
"sphinx-inline-tabs>=2023.4.21",
"sphinxcontrib-towncrier>=0.2.1a0",
Expand All @@ -82,7 +82,7 @@ optional-dependencies.testing = [
"hatch-vcs>=0.3",
"hatchling>=1.17.1",
"psutil>=5.9.5",
"pytest>=7.3.2",
"pytest>=7.4",
"pytest-cov>=4.1",
"pytest-mock>=3.11.1",
"pytest-xdist>=3.3.1",
Expand Down
4 changes: 2 additions & 2 deletions src/tox/config/cli/ini.py
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
import os
from configparser import ConfigParser
from pathlib import Path
from typing import Any
from typing import Any, ClassVar

from platformdirs import user_config_dir

Expand All @@ -18,7 +18,7 @@

class IniConfig:
TOX_CONFIG_FILE_ENV_VAR = "TOX_USER_CONFIG_FILE"
STATE = {None: "failed to parse", True: "active", False: "missing"}
STATE: ClassVar[dict[bool | None, str]] = {None: "failed to parse", True: "active", False: "missing"}

def __init__(self) -> None:
config_file = os.environ.get(self.TOX_CONFIG_FILE_ENV_VAR, None)
Expand Down
9 changes: 6 additions & 3 deletions src/tox/config/loader/str_convert.py
Original file line number Diff line number Diff line change
Expand Up @@ -5,11 +5,14 @@
import sys
from itertools import chain
from pathlib import Path
from typing import Any, Iterator
from typing import TYPE_CHECKING, Any, Iterator

from tox.config.loader.convert import Convert
from tox.config.types import Command, EnvList

if TYPE_CHECKING:
from typing import Final


class StrConvert(Convert[str]):
"""A class converting string values to tox types."""
Expand Down Expand Up @@ -111,8 +114,8 @@ def to_env_list(value: str) -> EnvList:
elements = list(chain.from_iterable(extend_factors(expr) for expr in value.split("\n")))
return EnvList(elements)

TRUTHFUL_VALUES = {"true", "1", "yes", "on"}
FALSE_VALUES = {"false", "0", "no", "off", ""}
TRUTHFUL_VALUES: Final[set[str]] = {"true", "1", "yes", "on"}
FALSE_VALUES: Final[set[str]] = {"false", "0", "no", "off", ""}
VALID_BOOL = sorted(TRUTHFUL_VALUES | FALSE_VALUES)

@staticmethod
Expand Down
4 changes: 2 additions & 2 deletions src/tox/report.py
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@
from io import BytesIO, TextIOWrapper
from pathlib import Path
from threading import Thread, current_thread, enumerate, local
from typing import IO, Iterator, Tuple
from typing import IO, ClassVar, Iterator, Tuple

from colorama import Fore, Style, init

Expand All @@ -29,7 +29,7 @@
class _LogThreadLocal(local):
"""A thread local variable that inherits values from its parent."""

_ident_to_data: dict[int | None, str] = {}
_ident_to_data: ClassVar[dict[int | None, str]] = {}

def __init__(self, out_err: OutErr) -> None:
self.name = self._ident_to_data.get(getattr(current_thread(), "parent_ident", None), "ROOT")
Expand Down
3 changes: 2 additions & 1 deletion src/tox/tox_env/python/pip/req_file.py
Original file line number Diff line number Diff line change
Expand Up @@ -8,12 +8,13 @@
if TYPE_CHECKING:
from argparse import ArgumentParser, Namespace
from pathlib import Path
from typing import Final


class PythonDeps(RequirementsFile):
# these options are valid in requirements.txt, but not via pip cli and
# thus cannot be used in the testenv `deps` list
_illegal_options = ["hash"]
_illegal_options: Final[list[str]] = ["hash"]

def __init__(self, raw: str, root: Path) -> None:
super().__init__(root / "tox.ini", constraint=False)
Expand Down
7 changes: 4 additions & 3 deletions src/tox/util/spinner.py
Original file line number Diff line number Diff line change
Expand Up @@ -13,12 +13,13 @@

if TYPE_CHECKING:
from types import TracebackType
from typing import Any, ClassVar

if sys.platform == "win32": # pragma: win32 cover
import ctypes

class _CursorInfo(ctypes.Structure):
_fields_ = [("size", ctypes.c_int), ("visible", ctypes.c_byte)]
_fields_: ClassVar[list[tuple[str, Any]]] = [("size", ctypes.c_int), ("visible", ctypes.c_byte)]


def _file_support_encoding(chars: Sequence[str], file: IO[str]) -> bool:
Expand Down Expand Up @@ -47,8 +48,8 @@ class Outcome(NamedTuple):
class Spinner:
CLEAR_LINE = "\033[K"
max_width = 120
UNICODE_FRAMES = ["⠋", "⠙", "⠹", "⠸", "⠼", "⠴", "⠦", "⠧", "⠇", "⠏"]
ASCII_FRAMES = ["|", "-", "+", "x", "*"]
UNICODE_FRAMES: ClassVar[list[str]] = ["⠋", "⠙", "⠹", "⠸", "⠼", "⠴", "⠦", "⠧", "⠇", "⠏"]
ASCII_FRAMES: ClassVar[list[str]] = ["|", "-", "+", "x", "*"]
UNICODE_OUTCOME = Outcome(ok="✔", fail="✖", skip="⚠")
ASCII_OUTCOME = Outcome(ok="+", fail="!", skip="?")

Expand Down
4 changes: 2 additions & 2 deletions tests/util/test_ci.py
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@
"TEAMCITY_VERSION": None, # TeamCity
"TRAVIS": "true", # Travis CI
}.items(),
ids=lambda v: v[0], # type: ignore[no-any-return]
ids=lambda v: v[0],
)
def test_is_ci(env_var: tuple[str, str | None], monkeypatch: pytest.MonkeyPatch) -> None:
for var in _ENV_VARS:
Expand All @@ -41,7 +41,7 @@ def test_is_ci(env_var: tuple[str, str | None], monkeypatch: pytest.MonkeyPatch)
"GITHUB_ACTIONS": "", # GitHub Actions
"TRAVIS": "", # Travis CI
}.items(),
ids=lambda v: v[0], # type: ignore[no-any-return]
ids=lambda v: v[0],
)
def test_is_ci_bad_set(env_var: tuple[str, str], monkeypatch: pytest.MonkeyPatch) -> None:
for var in _ENV_VARS:
Expand Down
2 changes: 1 addition & 1 deletion tox.ini
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,7 @@ commands =
[testenv:type]
description = run type check on code base
deps =
mypy==1.3
mypy==1.4.1
types-cachetools>=5.3.0.5
types-chardet>=5.0.4.6
commands =
Expand Down

0 comments on commit 67c5cb3

Please sign in to comment.