Update test deps (#3589)

Author: ssbarnea

.config/requirements-lock.txt

@@ -5,7 +5,7 @@
 #    pip-compile --no-annotate --output-file=.config/requirements-lock.txt --resolver=backtracking --strip-extras --unsafe-package=resolvelib --unsafe-package=ruamel-yaml-clib pyproject.toml
 #
 ansible-compat==4.1.2
-ansible-core==2.15.0
+ansible-core==2.15.1
 attrs==23.1.0
 black==23.3.0
 bracex==2.3.post1
@@ -25,7 +25,7 @@ mdurl==0.1.2
 mypy-extensions==1.0.0
 packaging==23.1
 pathspec==0.11.1
-platformdirs==3.6.0
+platformdirs==3.7.0
 pycparser==2.21
 pygments==2.15.1
 pyrsistent==0.19.3

.config/requirements.txt

@@ -5,7 +5,7 @@
 #    pip-compile --extra=docs --extra=test --no-annotate --output-file=.config/requirements.txt --resolver=backtracking --strip-extras --unsafe-package=resolvelib --unsafe-package=ruamel-yaml-clib pyproject.toml
 #
 ansible-compat==4.1.2
-ansible-core==2.15.0
+ansible-core==2.15.1
 astroid==2.15.5
 attrs==23.1.0
 beautifulsoup4==4.12.2
@@ -61,15 +61,15 @@ mkdocs-minify-plugin==0.6.4
 mkdocs-monorepo-plugin==1.0.5
 mkdocstrings==0.22.0
 mkdocstrings-python==1.1.0
-mypy==1.3.0
+mypy==1.4.0
 mypy-extensions==1.0.0
 netaddr==0.8.0
 packaging==23.1
 pathspec==0.11.1
 pillow==9.5.0
 pipdeptree==2.7.1
-platformdirs==3.6.0
-pluggy==1.0.0
+platformdirs==3.7.0
+pluggy==1.2.0
 ply==3.11
 psutil==5.9.5
 pycparser==2.21

.pre-commit-config.yaml

@@ -130,7 +130,7 @@ repos:
         types: [file, yaml]
         entry: yamllint --strict
   - repo: https://github.com/charliermarsh/ruff-pre-commit
-    rev: "v0.0.272"
+    rev: "v0.0.274"
     hooks:
       - id: ruff
         args: [--fix, --exit-non-zero-on-fix]
@@ -140,7 +140,7 @@ repos:
       - id: black
         language_version: python3
   - repo: https://github.com/pre-commit/mirrors-mypy
-    rev: v1.3.0
+    rev: v1.4.0
     hooks:
       - id: mypy
         # empty args needed in order to match mypy cli behavior

pyproject.toml

@@ -223,7 +223,7 @@ python_files = [
 xfail_strict = true
 
 [tool.ruff]
-required-version = "0.0.272"
+required-version = "0.0.274"
 ignore = [
   "E501", # we use black
   "ERA001", # auto-removal of commented out code affects development and vscode integration
@@ -237,6 +237,7 @@ ignore = [
   "FBT001",
   "FBT003",
   "PLR",
+  "RUF012", # Mutable class attributes should be annotated with `typing.ClassVar`
   "TRY",
 ]
 select = ["ALL"]

src/ansiblelint/_internal/rules.py

@@ -143,7 +143,7 @@ def match(self, line: str) -> bool | str:
 
     def __lt__(self, other: BaseRule) -> bool:
         """Enable us to sort rules by their id."""
-        return (self._order, self.id) < (other._order, other.id)  # noqa: SLF001
+        return (self._order, self.id) < (other._order, other.id)
 
     def __repr__(self) -> str:
         """Return a AnsibleLintRule instance representation."""

src/ansiblelint/errors.py

@@ -153,7 +153,7 @@ def __lt__(self, other: object) -> bool:
         """Return whether the current object is less than the other."""
         if not isinstance(other, self.__class__):
             return NotImplemented
-        return bool(self._hash_key < other._hash_key)  # noqa: SLF001
+        return bool(self._hash_key < other._hash_key)
 
     def __eq__(self, other: object) -> bool:
         """Identify whether the other object represents the same rule match."""

src/ansiblelint/rules/meta_video_links.py

@@ -69,7 +69,7 @@ def matchyaml(self, file: Lintable) -> list[MatchError]:
                 )
                 continue
 
-            for _, expr in self.VIDEO_REGEXP.items():
+            for expr in self.VIDEO_REGEXP.values():
                 if expr.match(video["url"]):
                     break
             else:

src/ansiblelint/skip_utils.py

@@ -263,7 +263,7 @@ def _get_rule_skips_from_yaml(
         return []
 
     def traverse_yaml(obj: Any) -> None:
-        for _, entry in obj.ca.items.items():
+        for entry in obj.ca.items.values():
             for v in entry:
                 if isinstance(v, CommentToken):
                     comment_str = v.value
@@ -278,7 +278,7 @@ def traverse_yaml(obj: Any) -> None:
                         )
         yaml_comment_obj_strings.append(str(obj.ca.items))
         if isinstance(obj, dict):
-            for _, val in obj.items():
+            for val in obj.values():
                 if isinstance(val, (dict, list)):
                     traverse_yaml(val)
         elif isinstance(obj, list):