Update pre-commit and hooks, lint accordingly

tekktrik · tekktrik · commit 97f29e6185bb · 2023-07-05T23:08:01.000-04:00
diff --git a/.pre-commit-config.yaml b/.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
diff --git a/.pylintrc b/.pylintrc
diff --git a/LICENSES/CC-BY-4.0.txt b/LICENSES/CC-BY-4.0.txt
diff --git a/adabot/circuitpython_bundle.py b/adabot/circuitpython_bundle.py
@@ -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")
diff --git a/adabot/circuitpython_libraries.py b/adabot/circuitpython_libraries.py
@@ -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"""
diff --git a/adabot/lib/circuitpython_library_validators.py b/adabot/lib/circuitpython_library_validators.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")]
                 )
diff --git a/tests/integration/test_circuitpython_libraries.py b/tests/integration/test_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."""
diff --git a/tests/integration/test_update_cp_org_libraries.py b/tests/integration/test_update_cp_org_libraries.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."""
diff --git a/tools/ci_status.py b/tools/ci_status.py
@@ -295,7 +295,6 @@ def save_build_statuses(
 
 
 if __name__ == "__main__":
-
     parser = argparse.ArgumentParser(
         description="Check the CI status of the Bundle libraries"
     )
diff --git a/tools/docs_status.py b/tools/docs_status.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"
     )
diff --git a/tools/file_compare.py b/tools/file_compare.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",
     )
diff --git a/tools/git_functionality.py b/tools/git_functionality.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)
 
diff --git a/tools/iterate_libraries.py b/tools/iterate_libraries.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
             )
diff --git a/tools/library_functions.py b/tools/library_functions.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()
 

Original file line number	Diff line number	Diff line change
`@@ -93,6 +93,7 @@`
`93`	`93`	`"Adafruit_Python_Extended_Bus",`
`94`	`94`	`]`
`95`	`95`
	`96`	`+`
`96`	`97`	`# pylint: disable=too-many-locals, too-many-branches, too-many-statements`
`97`	`98`	`def run_library_checks(validators, kw_args, error_depth):`
`98`	`99`	`"""runs the various library checking functions"""`
Original file line number	Diff line number	Diff line change
`@@ -295,7 +295,6 @@ def save_build_statuses(`
`295`	`295`
`296`	`296`
`297`	`297`	`if __name__ == "__main__":`
`298`		`-`
`299`	`298`	`parser = argparse.ArgumentParser(`
`300`	`299`	`description="Check the CI status of the Bundle libraries"`
`301`	`300`	`)`
Original file line number	Diff line number	Diff line change
`@@ -106,7 +106,6 @@ def check_docs_statuses(`
`106`	`106`
`107`	`107`
`108`	`108`	`if __name__ == "__main__":`
`109`		`-`
`110`	`109`	`parser = argparse.ArgumentParser(`
`111`	`110`	`description="Check the RTD docs build status of the Bundle libraries"`
`112`	`111`	`)`