build: upgrade Poetry, pylint and pre-commit hooks (#494)

* [pre-commit.ci] pre-commit autoupdate

updates:
- [github.com/pre-commit/pre-commit-hooks: v4.1.0 → v4.3.0](pre-commit/pre-commit-hooks@v4.1.0...v4.3.0)
- [github.com/asottile/pyupgrade: v2.31.1 → v2.37.1](asottile/pyupgrade@v2.31.1...v2.37.1)
- [github.com/psf/black: 22.1.0 → 22.6.0](psf/black@22.1.0...22.6.0)
- [github.com/pre-commit/mirrors-prettier: v2.6.1 → v2.7.1](pre-commit/mirrors-prettier@v2.6.1...v2.7.1)
- [github.com/pre-commit/mirrors-mypy: v0.942 → v0.961](pre-commit/mirrors-mypy@v0.942...v0.961)
- [github.com/commitizen-tools/commitizen: v2.21.2 → v2.28.0](commitizen-tools/commitizen@v2.21.2...v2.28.0)

* build: upgrade Poetry and pylint
* build: remove @lru_cache on methods to fix flake8 warning B019
* ci: use Python3.8 only on flake8
* ci: upgrade black and all hooks

black is failing on pre-commit.ci with
ImportError: cannot import name '_unicodefun' from 'click'

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

Author: andreoliwa

.pre-commit-config.yaml

@@ -34,13 +34,13 @@ repos:
         types: [python, toml]
         stages: [commit]
   - repo: https://github.com/pre-commit/pre-commit-hooks
-    rev: v4.1.0
+    rev: v4.3.0
     hooks:
       - id: debug-statements
       - id: end-of-file-fixer
       - id: trailing-whitespace
   - repo: https://github.com/asottile/pyupgrade
-    rev: v2.31.1
+    rev: v2.37.1
     hooks:
       - id: pyupgrade
         args: [--py37-plus]
@@ -64,7 +64,7 @@ repos:
             compat.py,
           ]
   - repo: https://github.com/psf/black
-    rev: 22.1.0
+    rev: 22.6.0
     hooks:
       - id: black
         args: [--safe, --quiet]
@@ -86,14 +86,15 @@ repos:
       - id: python-no-log-warn
       - id: rst-backticks
   - repo: https://github.com/pre-commit/mirrors-prettier
-    rev: v2.6.1
+    rev: v2.7.1
     hooks:
       - id: prettier
         stages: [commit]
   - repo: https://github.com/PyCQA/flake8
     rev: 4.0.1
     hooks:
       - id: flake8
+        language_version: python3.8
         additional_dependencies:
           [
             flake8-blind-except,
@@ -109,7 +110,7 @@ repos:
             yesqa,
           ]
   - repo: https://github.com/pre-commit/mirrors-mypy
-    rev: v0.942
+    rev: v0.961
     hooks:
       - id: mypy
         # https://mypy.readthedocs.io/en/stable/command_line.html#cmdoption-mypy-show-error-codes
@@ -154,7 +155,7 @@ repos:
         # https://docs.openstack.org/bashate/latest/man/bashate.html#options
         args: [-i, E006]
   - repo: https://github.com/commitizen-tools/commitizen
-    rev: v2.21.2
+    rev: v2.28.0
     hooks:
       - id: commitizen
         stages: [commit-msg]

.tool-versions

@@ -1,2 +1,2 @@
-poetry 1.1.12
+poetry 1.1.14
 pre-commit 2.17.0

poetry.lock

@@ -10,7 +10,7 @@
 import json
 import re
 import shlex
-from functools import lru_cache, partial
+from functools import partial
 from pathlib import Path
 from typing import Any, Callable, TypeVar, cast
 
@@ -494,7 +494,6 @@ def __init__(
         super().__init__(path=path, string=string, obj=obj)
         self.use_tomlkit = use_tomlkit
 
-    @lru_cache()
     def load(self) -> bool:
         """Load a TOML file by its path, a string or a dict."""
         if self.path is not None:
@@ -586,7 +585,6 @@ class YamlDoc(BaseDoc):
 
     updater: SensibleYAML
 
-    @lru_cache()
     def load(self) -> bool:
         """Load a YAML file by its path, a string or a dict."""
         self.updater = SensibleYAML()
@@ -663,7 +661,6 @@ def traverse_yaml_tree(yaml_obj: YamlObject, change: JsonDict):
 class JsonDoc(BaseDoc):
     """JSON configuration format."""
 
-    @lru_cache()
     def load(self) -> bool:
         """Load a JSON file by its path, a string or a dict."""
         if self.path is not None:

src/nitpick/blender.py

@@ -3,7 +3,6 @@
 
 import abc
 import fnmatch
-from functools import lru_cache
 from pathlib import Path
 from typing import Iterator
 
@@ -83,7 +82,6 @@ def predefined_special_config(self) -> SpecialConfig:  # pylint: disable=no-self
         return SpecialConfig()
 
     @mypy_property
-    @lru_cache()
     def nitpick_file_dict(self) -> JsonDict:
         """Nitpick configuration for this file as a TOML dict, taken from the style file."""
         return search_json(self.info.project.nitpick_section, f'files."{self.filename}"', {})

src/nitpick/plugins/base.py

@@ -161,8 +161,7 @@ class GitHubFetcher(HttpFetcher):  # pylint: disable=too-few-public-methods
     protocols: tuple[str, ...] = (Scheme.GH, Scheme.GITHUB)  # type: ignore
     domains: tuple[str, ...] = (GITHUB_COM,)
 
-    @staticmethod
-    def _normalize_scheme(scheme: str) -> str:
+    def _normalize_scheme(self, scheme: str) -> str:  # pylint: disable=no-self-use
         # Use github:// instead of gh:// in the canonical URL
         return Scheme.GITHUB if scheme == Scheme.GH else scheme  # type: ignore