chore: cleanup deprecated linters (#41)

* chore: use pyupgrade for syntax check

* chore: fix markdownlint

* chore: fix ruff linter and formatter

* chore: remove black, isort, autopep8 and flake8

* chore: update pre commit packages

Author: ideepu

.markdownlint.yaml

@@ -13,16 +13,18 @@ repos:
     -   id: check-builtin-literals
     -   id: check-docstring-first
     -   id: debug-statements
-    -   id: fix-encoding-pragma
     -   id: double-quote-string-fixer
     -   id: check-added-large-files
     -   id: check-merge-conflict
     -   id: name-tests-test
         # ensure tests match `test_.*\.py`
-        # NOTE: arg should be `--pytest-test-first` but only the following works now
-        args: ['--django']
+        args: ['--pytest-test-first']
     -   id: no-commit-to-branch
-        args: [--branch, master]
+        args: [--branch, main]
+-   repo: https://github.com/asottile/pyupgrade
+    rev: v3.19.1
+    hooks:
+    -   id: pyupgrade
 -   repo: https://github.com/pre-commit/pygrep-hooks
     rev: v1.10.0
     hooks:
@@ -33,49 +35,30 @@ repos:
     -   id: python-no-eval
     -   id: python-no-log-warn
 -   repo: https://github.com/igorshubovych/markdownlint-cli
-    rev: v0.38.0
+    rev: v0.44.0
     hooks:
-    -   id: markdownlint-fix
-        args: ['--config', '.markdownlint.yaml']
+    -   id: markdownlint
+        args: ['--config', 'pyproject.toml', '--fix']
 - repo: https://github.com/astral-sh/ruff-pre-commit
-  rev: v0.1.6
+  rev: v0.11.2
   hooks:
     - id: ruff
     - id: ruff-format
       args: ['--check']
--   repo: https://github.com/psf/black-pre-commit-mirror
-    rev: 23.11.0
-    hooks:
-    -   id: black
-        args: ['--check']
--   repo: https://github.com/hhatto/autopep8
-    rev: v2.0.4
-    hooks:
-    -   id: autopep8
 -   repo: https://github.com/pre-commit/mirrors-mypy
-    rev: v1.7.0
+    rev: v1.15.0
     hooks:
     -   id: mypy
         additional_dependencies: ['types-pytz', 'types-requests']
--   repo: https://github.com/PyCQA/flake8
-    rev: 6.1.0
-    hooks:
-    -   id: flake8
-        additional_dependencies: ['Flake8-pyproject==1.2.3']
 -   repo: https://github.com/PyCQA/bandit
-    rev: 1.7.5
+    rev: 1.8.3
     hooks:
     -   id: bandit
         args: ["-c", "pyproject.toml"]
 -   repo: https://github.com/pylint-dev/pylint
-    rev: v3.0.2
+    rev: v3.3.6
     hooks:
     -   id: pylint
--   repo: https://github.com/PyCQA/isort
-    rev: 5.12.0
-    hooks:
-    -   id: isort
-        args: ['--check-only']
 -   repo: https://github.com/hadialqattan/pycln
     rev: v2.5.0
     hooks:
@@ -85,7 +68,7 @@ repos:
     hooks:
     -   id: checkmake
 - repo: https://github.com/tox-dev/pyproject-fmt
-  rev: "1.0.0"
+  rev: v2.5.1
   hooks:
     - id: pyproject-fmt
       args: ['--check']

.pre-commit-config.yaml

@@ -9,7 +9,6 @@ httpx = "*"
 jinja2 = "*"
 
 [dev-packages]
-ruff = "*"
 mypy = "*"
 pytest = "*"
 pytest-env = "*"
@@ -19,6 +18,7 @@ setuptools = "*"
 build = "*"
 pre-commit = "*"
 twine = "*"
+ruff = "*"
 
 [requires]
 python_version = "3.11"

Pipfile

@@ -1,4 +1,3 @@
-# -*- coding: utf-8 -*-
 """
 This module should contain only the things relevant to the badge being computed
 by shields.io

Pipfile.lock

@@ -1,4 +1,3 @@
-# -*- coding: utf-8 -*-
 from .base import Coverage, CoverageInfo, CoverageMetadata, DiffCoverage, FileCoverage, FileDiffCoverage
 from .pytest import PytestCoverage
 

codecov/badge.py

@@ -1,4 +1,3 @@
-# -*- coding: utf-8 -*-
 from __future__ import annotations
 
 import dataclasses

codecov/coverage/__init__.py

@@ -1,4 +1,3 @@
-# -*- coding: utf-8 -*-
 from __future__ import annotations
 
 import datetime

codecov/coverage/base.py

@@ -1,4 +1,3 @@
-# -*- coding: utf-8 -*-
 from __future__ import annotations
 
 from collections.abc import Iterable

codecov/coverage/pytest.py

@@ -1,4 +1,3 @@
-# -*- coding: utf-8 -*-
 class CoreBaseException(Exception):
     pass
 

codecov/diff_grouper.py

@@ -1,4 +1,3 @@
-# -*- coding: utf-8 -*-
 import dataclasses
 
 from codecov.exceptions import (
@@ -27,7 +26,7 @@ class User:
 
 
 class Github:
-    def __init__(  # pylint: disable=too-many-arguments
+    def __init__(  # pylint: disable=too-many-arguments, too-many-positional-arguments
         self,
         client: GitHubClient,
         repository: str,

codecov/exceptions.py

@@ -1,4 +1,3 @@
-# -*- coding: utf-8 -*-
 from __future__ import annotations
 
 import httpx

codecov/github.py

@@ -1,4 +1,3 @@
-# -*- coding: utf-8 -*-
 from __future__ import annotations
 
 import base64

codecov/github_client.py

@@ -1,4 +1,3 @@
-# -*- coding: utf-8 -*-
 import logging
 
 log = logging.getLogger('codecov')

codecov/groups.py

@@ -1,4 +1,3 @@
-# -*- coding: utf-8 -*-
 import json
 import os
 

codecov/log.py

@@ -1,12 +1,11 @@
-# -*- coding: utf-8 -*-
 from __future__ import annotations
 
 import dataclasses
 import decimal
 import inspect
 import pathlib
 from collections.abc import MutableMapping
-from typing import Any
+from typing import Any, Callable
 
 from codecov.exceptions import InvalidAnnotationType, MissingEnvironmentVariable
 
@@ -116,7 +115,8 @@ def from_environ(cls, environ: MutableMapping[str, str]) -> Config:
         possible_variables = list(inspect.signature(cls).parameters)
         config_dict: dict[str, Any] = {k: v for k, v in environ.items() if k in possible_variables}
         for key, value in list(config_dict.items()):
-            if func := getattr(cls, f'clean_{key.lower()}', None):
+            if hasattr(cls, f'clean_{key.lower()}'):
+                func: Callable = getattr(cls, f'clean_{key.lower()}')
                 try:
                     config_dict[key] = func(value)
                 except ValueError as exc:
@@ -130,5 +130,5 @@ def from_environ(cls, environ: MutableMapping[str, str]) -> Config:
                 for name, param in inspect.signature(cls).parameters.items()
                 if param.default is inspect.Parameter.empty
             } - set(environ)
-            raise MissingEnvironmentVariable(f" missing environment variable(s): {', '.join(missing)}") from e
+            raise MissingEnvironmentVariable(f' missing environment variable(s): {", ".join(missing)}') from e
         return config_obj

codecov/main.py

@@ -1,4 +1,3 @@
-# -*- coding: utf-8 -*-
 from __future__ import annotations
 
 import dataclasses
@@ -239,7 +238,7 @@ def get_file_url(  # pylint: disable=too-many-arguments
         return s
 
     # To link to a file in a PR, GitHub uses the link to the file overview combined with a SHA256 hash of the file path
-    s = f"https://github.com/{repo_name}/pull/{pr_number}/files#diff-{hashlib.sha256(str(filename).encode('utf-8')).hexdigest()}"
+    s = f'https://github.com/{repo_name}/pull/{pr_number}/files#diff-{hashlib.sha256(str(filename).encode("utf-8")).hexdigest()}'
 
     if lines is not None:
         # R stands for Right side of the diff. But since we generate these links for new code we only need the right side.

codecov/settings.py

@@ -1,4 +1,19 @@
+[build-system]
+build-backend = "setuptools.build_meta"
+requires = [
+  "setuptools",
+]
+
 [project]
+name = "python-coverage-comment"
+version = "2.1"
+description = "Create a Coverage report comment on Github PR"
+readme = "README.md"
+license = { file = "LICENSE" }
+maintainers = [
+  { name = "Pradeep Tammali", email = "[email protected]" },
+]
+requires-python = ">=3.0"
 classifiers = [
   "Programming Language :: Python :: 3 :: Only",
   "Programming Language :: Python :: 3.0",
@@ -13,89 +28,39 @@ classifiers = [
   "Programming Language :: Python :: 3.9",
   "Programming Language :: Python :: 3.10",
   "Programming Language :: Python :: 3.11",
+  "Programming Language :: Python :: 3.12",
+  "Programming Language :: Python :: 3.13",
 ]
 dependencies = [
   "httpx",
   "jinja2",
 ]
-description = "Create a Coverage report comment on Github PR"
-license = {file = "LICENSE"}
-maintainers = [
-  {name = "Pradeep Tammali", email = "[email protected]"},
-]
-name = "python-coverage-comment"
-readme = "README.md"
-requires-python = ">=3.0"
-version = "2.1"
-
-[project.urls]
-Homepage = "https://github.com/PradeepTammali/python-coverage-comment"
-Issues = "https://github.com/PradeepTammali/python-coverage-comment/issues"
-[build-system]
-build-backend = "setuptools.build_meta"
-requires = [
-  "setuptools",
-]
+urls.Homepage = "https://github.com/PradeepTammali/python-coverage-comment"
+urls.Issues = "https://github.com/PradeepTammali/python-coverage-comment/issues"
+scripts.codecov = "codecov.main:main"
 
-[project.scripts]
-codecov = "codecov.main:main"
-[tool.autopep8]
-in-place = false
-max_line_length = 120
+[tool.ruff]
+target-version = "py311"
 
-[tool.black]
 line-length = 120
-skip-string-normalization = true
-
-[tool.ruff]
-ignore = [
-  # format
-  'E501', # line too long
+format.quote-style = "single"
+lint.select = [
+  "E", # Errors
+  "F", # Formatting
+  "S", # Bandit
+  "W", # Warnings
 ]
-line-length = 120
-select = [
-  'E', # Errors
-  'W', # Warnings
-  'F', # Formatting
-  'S', # Bandit
+lint.ignore = [
+  # format
+  "E501", # line too long
 ]
-target-version = 'py311'
-
-[tool.ruff.mccabe]
-max-complexity = 10
-
-[tool.ruff.lint.isort]
-combine-as-imports = true
-
-[tool.ruff.format]
-quote-style = 'single'
-
-[tool.ruff.per-file-ignores]
 # Ignore assert usage in tests
-'tests/*.py' = ['S101']
-
-[tool.isort]
-combine_as_imports = true
-line_length = 120
-profile = "black"
-
-[tool.flake8]
-ignore = ['E501', 'W503', 'W504']
-
-[tool.pytest.ini_options]
-env = ['APP_ENVIRONMENT = unittest']
-
-[tool.coverage.report]
-show_missing = true
-
-[[tool.mypy.overrides]]
-check_untyped_defs = true
-ignore_missing_imports = true
-implicit_optional = true
-module = 'codecov.*'
+lint.per-file-ignores.'tests/*.py' = [ "S101" ]
+lint.isort.combine-as-imports = true
+lint.mccabe.max-complexity = 10
 
 [tool.pylint.MASTER]
-ignore-paths = ['tests/*']
+ignore-paths = [ 'tests/*' ]
 
 [tool.pylint.FORMAT]
 indent-string = '    '
@@ -111,5 +76,31 @@ disable = '''
     too-few-public-methods
 '''
 
+[tool.pytest.ini_options]
+env = [ 'APP_ENVIRONMENT = unittest' ]
+
+[tool.coverage.report]
+show_missing = true
+
+[[tool.mypy.overrides]]
+check_untyped_defs = true
+ignore_missing_imports = true
+implicit_optional = true
+module = 'codecov.*'
+
+[tool.markdownlint]
+default = true
+
+[MD007]
+indent = 4
+
+[MD013]
+# Number of characters
+line_length = 120
+# Number of characters for headings
+heading_line_length = 80
+# Number of characters for code blocks
+code_block_line_length = 120
+
 [tool.bandit]
-exclude_dirs = ["tests/*"]
+exclude_dirs = [ "tests/*" ]

codecov/template.py

@@ -1,4 +1,3 @@
-# -*- coding: utf-8 -*-
 from codecov.main import Main
 
 

pyproject.toml

@@ -1,4 +1,3 @@
-# -*- coding: utf-8 -*-
 # pylint: disable=redefined-outer-name
 # mypy: disable-error-code="operator, union-attr"
 from __future__ import annotations