[pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

Author: pre-commit-ci[bot]

BWBImportBot/__init__.py

@@ -1,10 +1,10 @@
 #!/usr/bin/env python
 
 """
-    __init__.py
-    ~~~~~~~~~~~
-    :copyright: (c) 2020 by Internet Archive.
-    :license: AGPL v3
+__init__.py
+~~~~~~~~~~~
+:copyright: (c) 2020 by Internet Archive.
+:license: AGPL v3
 """
 
 __title__ = "bwbimportbot"

ia-bulkmarc-bot/bulk-import.py

@@ -1,12 +1,12 @@
 #!/usr/bin/env python3
 
 """
-    Iterates over an archive.org bulk MARC item, such as OpenLibraries-Trent-MARCs,
-    and imports all records in all of its MARC files to Open Library.
+Iterates over an archive.org bulk MARC item, such as OpenLibraries-Trent-MARCs,
+and imports all records in all of its MARC files to Open Library.
 
-    USAGE: ./bulk-import.py <archive.org item id>
+USAGE: ./bulk-import.py <archive.org item id>
 
-    Logs results to STDOUT
+Logs results to STDOUT
 """
 
 import argparse

old-onix-bot/onix.py

@@ -105,7 +105,7 @@ def startElementNS(self, name, qname, attrs):
             self.subhandler.startElementNS(name, qname, attrs)
             self.subdepth += 1
         else:
-            (uri, localname) = name
+            uri, localname = name
             if localname == "product":
                 self.subhandler = xmltramp.Seeder(self.parser)
                 self.subhandler.startElementNS(name, qname, attrs)

old-onix-bot/sax_utils.py

@@ -121,7 +121,7 @@ def startElementNS(self, name, qname, attrs):
         if self.ignoring:
             self.ignoring += 1
         else:
-            (uri, localname) = name
+            uri, localname = name
             c_maker = self.collector_table.get(localname) or self.collector_table.get(
                 collector_any
             )
@@ -180,7 +180,7 @@ def __init__(self, collector_table):
         self.values = {}
 
     def collect(self, key_value):
-        (key, value) = key_value
+        key, value = key_value
         if self.values.get(key):
             raise Exception(f"dictionary key '{key}' is already mapped")
         else:

old-onix-bot/thread_utils.py

@@ -43,7 +43,7 @@ def re_raise(self):
 
 
 def ForeignException_extract():
-    (exc_type, exc_value, exc_traceback) = sys.exc_info()
+    exc_type, exc_value, exc_traceback = sys.exc_info()
     return ForeignException(exc_type, exc_value, exc_traceback)
 
 

old-onix-bot/urlcache.py

@@ -27,7 +27,7 @@ def get_entries(self):
 
     def get(self, url):
         url = url.strip()
-        (entries, next, index) = self.get_entries()
+        entries, next, index = self.get_entries()
         id = entries.get(url)
         if id is None:
             # with index locked, add an entry for this url and

old-onix-bot/xmltramp.py

@@ -415,17 +415,15 @@ def unittest():
     doc = Namespace("http://example.org/bar")
     bbc = Namespace("http://example.org/bbc")
     dc = Namespace("http://purl.org/dc/elements/1.1/")
-    d = parse(
-        """<doc version="2.7182818284590451"
+    d = parse("""<doc version="2.7182818284590451"
       xmlns="http://example.org/bar"
       xmlns:dc="http://purl.org/dc/elements/1.1/"
       xmlns:bbc="http://example.org/bbc">
             <author>John Polk and John Palfrey</author>
             <dc:creator>John Polk</dc:creator>
             <dc:creator>John Palfrey</dc:creator>
             <bbc:show bbc:station="4">Buffy</bbc:show>
-    </doc>"""
-    )
+    </doc>""")
 
     assert repr(d) == '<doc version="2.7182818284590451">...</doc>'
     assert (

pyproject.toml

@@ -1,26 +1,6 @@
 [tool.ruff]
-select = [
-  "C4",      # flake8-comprehensions
-  "C90",     # mccabe
-  "E",       # pycodestyle
-  "F",       # Pyflakes
-  "I",       # isort
-  "ICN",     # flake8-import-conventions
-  "PGH",     # pygrep-hooks
-  "PIE",     # flake8-pie
-  "PLC",     # Pylint conventions
-  "PLE",     # Pylint errors
-  "PLR091",  # Pylint refactor just for max-args, max-branches, etc.
-  "PYI",     # flake8-pyi
-  "RSE",     # flake8-raise
-  "RUF",     # Ruff-specific rules
-  "T10",     # flake8-debugger
-  "TCH",     # flake8-type-checking
-  "TID",     # flake8-tidy-imports
-  "UP",      # pyupgrade
-  "W",       # pycodestyle
-  "YTT",     # flake8-2020
-]
+target-version = "py37"
+line-length = 616
 ignore = [
   "E402",
   "E722",
@@ -31,20 +11,34 @@ ignore = [
   "PLC1901",
   "RUF001",
 ]
-line-length = 616
-target-version = "py37"
-
-[tool.ruff.mccabe]
-max-complexity = 21
-
-[tool.ruff.pylint]
-max-args = 8
-max-branches = 18
-max-statements = 73
-
-[tool.ruff.per-file-ignores]
-"test/*" = ["S101"]
+mccabe.max-complexity = 21
+per-file-ignores."test/*" = [ "S101" ]
+pylint.max-args = 8
+pylint.max-branches = 18
+pylint.max-statements = 73
+select = [
+  "C4",     # flake8-comprehensions
+  "C90",    # mccabe
+  "E",      # pycodestyle
+  "F",      # Pyflakes
+  "I",      # isort
+  "ICN",    # flake8-import-conventions
+  "PGH",    # pygrep-hooks
+  "PIE",    # flake8-pie
+  "PLC",    # Pylint conventions
+  "PLE",    # Pylint errors
+  "PLR091", # Pylint refactor just for max-args, max-branches, etc.
+  "PYI",    # flake8-pyi
+  "RSE",    # flake8-raise
+  "RUF",    # Ruff-specific rules
+  "T10",    # flake8-debugger
+  "TCH",    # flake8-type-checking
+  "TID",    # flake8-tidy-imports
+  "UP",     # pyupgrade
+  "W",      # pycodestyle
+  "YTT",    # flake8-2020
+]
 
 [tool.codespell]
-ignore-words-list = 'didnt'
-skip = './.*,*.xsd'
+ignore-words-list = "didnt"
+skip = "./.*,*.xsd"