Update pre-commit and hooks, lint accordingly

Author: tekktrik

.pre-commit-config.yaml

@@ -5,20 +5,20 @@
 exclude: patches
 repos:
 -   repo: https://github.com/pre-commit/pre-commit-hooks
-    rev: v4.0.1
+    rev: v4.4.0
     hooks:
     -   id: end-of-file-fixer
     -   id: trailing-whitespace
 -   repo: https://github.com/fsfe/reuse-tool
-    rev: v0.12.1
+    rev: v1.1.2
     hooks:
     -   id: reuse
 -   repo: https://github.com/psf/black
-    rev: 22.3.0
+    rev: 23.3.0
     hooks:
     -   id: black
 -   repo: https://github.com/pycqa/pylint
-    rev: v2.9.3
+    rev: v2.17.4
     hooks:
     -   id: pylint
         name: pylint

.pylintrc

@@ -113,7 +113,6 @@ def update_download_stats(bundle_path):
     with open(
         os.path.join(bundle_path, "circuitpython_library_pypi_stats.md"), "w"
     ) as md_file:
-
         # Write headers
         md_file.write("\n".join(lib_list_header))
         md_file.write("\n")

LICENSES/CC-BY-4.0.txt

@@ -93,6 +93,7 @@
     "Adafruit_Python_Extended_Bus",
 ]
 
+
 # pylint: disable=too-many-locals, too-many-branches, too-many-statements
 def run_library_checks(validators, kw_args, error_depth):
     """runs the various library checking functions"""

adabot/circuitpython_bundle.py

@@ -198,6 +198,7 @@ def uses_token(func):
     return func
 
 
+# pylint: disable=too-many-instance-attributes
 class LibraryValidator:
     """Class to hold instance variables needed to traverse the calling
     code, and the validator functions.
@@ -367,7 +368,6 @@ def _filter_file_diffs(filenames):
                 ".pre-commit-config.yaml",
                 ".pylintrc",
                 ".gitignore",
-                "CODE_OF_CONDUCT.md",
                 "README.rst",
                 "pyproject.toml",
             }
@@ -414,7 +414,6 @@ def _filter_file_diffs(filenames):
         compare_tags_json = compare_tags.json()
         if "status" in compare_tags_json:
             if compare_tags_json["status"] != "identical":
-
                 filtered_files = _filter_file_diffs(
                     [file["filename"] for file in compare_tags_json.get("files")]
                 )

adabot/circuitpython_libraries.py

@@ -12,6 +12,7 @@
 
 from adabot.lib import circuitpython_library_validators
 
+
 # pylint: disable=unused-argument
 def mock_list_repos(*args, **kwargs):
     """Function to monkeypatch `common_funcs.list_repos()` for a shorter set of repos."""

adabot/lib/circuitpython_library_validators.py

@@ -12,6 +12,7 @@
 from adabot import github_requests
 from adabot import update_cp_org_libraries
 
+
 # pylint: disable=unused-argument
 def mock_list_repos(*args, **kwargs):
     """Function to monkeypatch `common_funcs.list_repos()` for a shorter set of repos."""

tests/integration/test_circuitpython_libraries.py

@@ -295,7 +295,6 @@ def save_build_statuses(
 
 
 if __name__ == "__main__":
-
     parser = argparse.ArgumentParser(
         description="Check the CI status of the Bundle libraries"
     )

tests/integration/test_update_cp_org_libraries.py

@@ -106,7 +106,6 @@ def check_docs_statuses(
 
 
 if __name__ == "__main__":
-
     parser = argparse.ArgumentParser(
         description="Check the RTD docs build status of the Bundle libraries"
     )

tools/ci_status.py

@@ -83,7 +83,6 @@ def compare(git_file: str, token: Optional[str] = None) -> list:
 
 
 if __name__ == "__main__":
-
     parser = argparse.ArgumentParser(
         description="Compare files across the adafruit CircuitPython repos",
     )

tools/docs_status.py

@@ -132,11 +132,9 @@ def sync_commit_push(
     """
 
     def decorator_sync_commit_push(func):
-
         functools.wraps(func)
 
         def wrapper_sync_commit_push(lib_path: StrPath, *args, **kwargs) -> Any:
-
             # Fetch and pull to repo
             sync_and_checkout(lib_path, remote_name, branch_name)
 

tools/file_compare.py

@@ -107,13 +107,11 @@ def iter_local_bundle_with_func(
 
     # Loop through each bundle branch
     for branch_name in _BUNDLE_BRANCHES:
-
         libraries_glob_path = os.path.join(bundle_path, "libraries", branch_name, "*")
         libraries_path_list = glob.glob(libraries_glob_path)
 
         # Enter each library in the bundle
         for library_path in libraries_path_list:
-
             iterated.add(os.path.split(library_path)[1].lower())
             func_results = perform_func(library_path, func_workflow)
 
@@ -164,13 +162,11 @@ def iter_remote_bundle_with_func(
 
     # Loop through each bundle branch
     for branch_name in _BUNDLE_BRANCHES:
-
         branch_repos_path = "/".join(("libraries", branch_name))
         branch_repos: list[ContentFile] = bundle_repo.get_contents(branch_repos_path)
 
         # Enter each library in the bundle
         for repo_file in branch_repos:
-
             repo_name_result: parse.Result = parse.search(
                 "repos/adafruit/{repo_name:w}/", repo_file.git_url
             )

tools/git_functionality.py

@@ -24,6 +24,7 @@
 StrPath: TypeAlias = Union[str, os.PathLike[str]]
 """Path or path-like strings"""
 
+
 # pylint: disable=too-few-public-methods
 class LocalLibFunc(Protocol):
     """Typing protocol for methods (or callables) that take the following
@@ -65,7 +66,6 @@ def in_lib_path(func: LocalLibFunc) -> LocalLibFunc:
 
     @functools.wraps(func)
     def wrapper_use_lib_path(lib_path: StrPath, *args, **kwargs) -> Any:
-
         # Get the current directory
         current_path = os.getcwd()