use ruff, remove pylint config

Author: FredHappyface

.pre-commit-config.yaml

@@ -1,13 +1,13 @@
 repos:
   - repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: v0.2.2
+    rev: v0.3.3
     hooks:
       - id: ruff
         args: [ --fix ]
       - id: ruff-format
 
   - repo: https://github.com/RobertCraigie/pyright-python
-    rev: v1.1.351
+    rev: v1.1.354
     hooks:
     - id: pyright
 

documentation/reference/mutablezip/index.md

@@ -13,7 +13,7 @@
 
 ## MutableZipFile
 
-[Show source in __init__.py:13](../../../mutablezip/__init__.py#L13)
+[Show source in __init__.py:14](../../../mutablezip/__init__.py#L14)
 
 Add delete (via remove_file) and update (via writestr and write methods)
 To enable update features use MutableZipFile with the 'with statement',
@@ -35,9 +35,9 @@ class MutableZipFile(ZipFile):
 
 ### MutableZipFile()._closeAllTempFiles
 
-[Show source in __init__.py:121](../../../mutablezip/__init__.py#L121)
+[Show source in __init__.py:118](../../../mutablezip/__init__.py#L118)
 
-close all temporary files
+Close all temporary files
 
 #### Signature
 
@@ -47,9 +47,9 @@ def _closeAllTempFiles(self): ...
 
 ### MutableZipFile().removeFile
 
-[Show source in __init__.py:127](../../../mutablezip/__init__.py#L127)
+[Show source in __init__.py:124](../../../mutablezip/__init__.py#L124)
 
-flag a file with a delete marker
+Flag a file with a delete marker
 
 #### Signature
 
@@ -59,7 +59,7 @@ def removeFile(self, path: str | PathLike[str]): ...
 
 ### MutableZipFile().write
 
-[Show source in __init__.py:74](../../../mutablezip/__init__.py#L74)
+[Show source in __init__.py:73](../../../mutablezip/__init__.py#L73)
 
 #### Signature
 
@@ -75,7 +75,7 @@ def write(
 
 ### MutableZipFile().writestr
 
-[Show source in __init__.py:45](../../../mutablezip/__init__.py#L45)
+[Show source in __init__.py:44](../../../mutablezip/__init__.py#L44)
 
 #### Signature
 

mutablezip/__init__.py

@@ -1,4 +1,5 @@
-""" mutable zip file """
+"""mutable zip file"""
+
 from __future__ import annotations
 
 from os import PathLike
@@ -28,9 +29,7 @@ def __init__(
 		compression: int = ZIP_STORED,
 		allowZip64: bool = False,
 	):
-		super().__init__(
-			file, mode=mode, compression=compression, allowZip64=allowZip64
-		)
+		super().__init__(file, mode=mode, compression=compression, allowZip64=allowZip64)
 		# track file to override in zip
 		self._replace = {}
 		# Whether the with statement was called
@@ -83,9 +82,7 @@ def write(
 		# mark the entry, and create a temp-file for it
 		# we allow this only if the with statement is used
 		if self._allowUpdates and arcname in self.namelist():
-			tempFile = self._replace[arcname] = self._replace.get(
-				arcname, TemporaryFile()
-			)
+			tempFile = self._replace[arcname] = self._replace.get(arcname, TemporaryFile())
 			with open(filename, "rb") as source:
 				copyfileobj(source, tempFile)
 		# Behave normally
@@ -119,13 +116,13 @@ def __exit__(
 			self._allowUpdates = False
 
 	def _closeAllTempFiles(self):
-		"""close all temporary files"""
+		"""Close all temporary files"""
 		for tempFile in self._replace.values():
 			if hasattr(tempFile, "close"):
 				tempFile.close()
 
 	def removeFile(self, path: str | PathLike[str]):
-		"""flag a file with a delete marker"""
+		"""Flag a file with a delete marker"""
 		self._replace[path] = self.DeleteMarker()
 
 	def _rebuildZip(self):

pyproject.toml

@@ -22,47 +22,52 @@ readme = "README.md"
 [tool.poetry.dependencies]
 python = "^3.8"
 
-
 [tool.poetry.group.dev.dependencies]
-pytest = "^8.0.2"
+pytest = "^8.1.1"
 pylint = "^3.1.0"
 handsdown = "^2.1.0"
-coverage = "^7.4.3"
-
-[tool.ruff.format]
-indent-style = "tab"
-docstring-code-format = true
-line-ending = "lf"
-
-
-[tool.isort]
-profile = "black"
-indent = "Tab"
-
-[tool.pydocstyle]
-convention = "google"
-ignore = "D205,D415"
+coverage = "^7.4.4"
 
 [build-system]
 requires = ["poetry-core"]
 build-backend = "poetry.core.masonry.api"
 
-[tool.pylint.basic]
-argument-naming-style = "camelCase"
-attr-naming-style = "camelCase"
-function-naming-style = "camelCase"
-method-naming-style = "camelCase"
-variable-naming-style = "camelCase"
+[tool.ruff]
+line-length = 100
+indent-width = 4
+target-version = "py38"
 
-[tool.pylint.format]
-indent-string = "\t"
+[tool.ruff.lint]
+select = ["ALL"]
+ignore = [
+	"ANN101",  # type annotation for self in method
+	"COM812",  # enforce trailing comma
+	"D2",  # pydocstyle formatting
+	"ISC001",
+	"N",  # pep8 naming
+	"PLR09",  # pylint refactor too many
+	"TCH",  # type check blocks
+	"W191"  # ignore this to allow tabs
+]
+fixable = ["ALL"]
+
+[tool.ruff.lint.per-file-ignores]
+"**/{tests,docs,tools}/*" = ["D", "S101", "E402"]
+
+[tool.ruff.lint.flake8-tidy-imports]
+ban-relative-imports = "all" # Disallow all relative imports.
+
+[tool.ruff.format]
+indent-style = "tab"
+docstring-code-format = true
+line-ending = "lf"
 
-[tool.pylint.master]
-ignore-paths = ["tests"]
+[tool.pyright]
+venvPath = "."
+venv = ".venv"
 
-[tool.pylint.messages_control]
-enable = ["F", "E", "W", "R", "C"]
-disable = ["pointless-string-statement", "superfluous-parens"]
+[tool.coverage.run]
+branch = true
 
 [tool.tox]
 legacy_tox_ini = """

tests/test_main.py

@@ -1,6 +1,7 @@
 #!/usr/bin/env python3.9
 
-""" tests """
+"""tests"""
+
 from __future__ import annotations
 
 import sys
@@ -14,9 +15,7 @@
 
 
 def test_MutableZipFile():
-	with MutableZipFile(
-		f"{THISDIR}/data/mutable.zip", "a", compression=ZIP_DEFLATED
-	) as zipFile:
+	with MutableZipFile(f"{THISDIR}/data/mutable.zip", "a", compression=ZIP_DEFLATED) as zipFile:
 		with zipFile.open("foo.txt", "r") as file:
 			lines = [line.strip() for line in file.readlines()]
 		lines.append(b"new line")
@@ -26,9 +25,7 @@ def test_MutableZipFile():
 
 
 def test_ZipFile():
-	with ZipFile(
-		f"{THISDIR}/data/immutable.zip", "a", compression=ZIP_DEFLATED
-	) as zipFile:
+	with ZipFile(f"{THISDIR}/data/immutable.zip", "a", compression=ZIP_DEFLATED) as zipFile:
 		with zipFile.open("foo.txt", "r") as file:
 			lines = [line.strip() for line in file.readlines()]
 		lines.append(b"new line")