[chore] pre-commit autoupdate + lint

closes #286

Related: odoo/upgrade#7882
Signed-off-by: Christophe Simonis (chs) <[email protected]>

Author: KangOl

.pre-commit-config.yaml

@@ -14,15 +14,15 @@ repos:
         files: '^src/\w+/(tests|0\.0\.0)/.*\.py$'
 
   - repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: v0.11.3
+    rev: v0.12.0
     hooks:
     - id: ruff
       name: Check code with Ruff, apply automatic fixes
       args: [ --exit-non-zero-on-fix ]
     - id: ruff-format
       name: Format code with Ruff
   - repo: https://github.com/crate-ci/typos
-    rev: v1.31.1
+    rev: v1.33.1
     hooks:
       - id: typos
   - repo: https://github.com/pre-commit/pre-commit-hooks

pyproject.toml

@@ -116,6 +116,8 @@ convention = "pep257"
 # and for upgrade scripts
 "src/*/*/{pre,post,end}-*.py" = ["D"]
 
+[tool.ruff.per-file-target-version]
+"tools/fetch-release-notes-video-id.py" = "py312"
 
 [tool.typos.files]
 extend-exclude = [

src/base/0.0.0/end-no-respawn-fields.py

@@ -22,14 +22,12 @@ def migrate(cr, version):
     execute_values(
         cr._obj,
         "INSERT INTO no_respawn(model, field) VALUES %s",
-        # fmt:off
         [
             (model, field)
             for model, fields in util.ENVIRON["__renamed_fields"].items()
             for field, new_name in fields.items()
             if new_name is None  # means removed :p
         ],
-        # fmt:on
     )
     cr.execute(
         """

src/util/fields.py

@@ -1078,7 +1078,7 @@ def adapter(leaf, _or, _neg):
 
 
 def is_field_anonymized(cr, model, field):
-    from .modules import module_installed
+    from .modules import module_installed  # noqa: PLC0415
 
     _validate_model(model)
     if not module_installed(cr, "anonymization"):

src/util/helpers.py

@@ -155,7 +155,7 @@ def _ir_values_value(cr, prefix=None):
     cache = getattr(_ir_values_value, "cache", None)
 
     if cache is None:
-        from .pg import column_type
+        from .pg import column_type  # noqa: PLC0415
 
         if column_type(cr, "ir_values", "value") == "bytea":
             cr.execute("SELECT character_set_name FROM information_schema.character_sets")

src/util/orm.py

@@ -1,4 +1,5 @@
 # -*- coding: utf-8 -*-
+# ruff: noqa: PLC0415
 """
 Utility functions to perform operations via the ORM.
 

src/util/pg.py

@@ -637,7 +637,7 @@ def alter_column_type(cr, table, column, type, using=None, where=None, logger=_l
             logger.info("Column %r of table %r is already defined as %r", column, table, type)
             return
 
-    # remove the existing linked `ir_model_fields_selection` recods in case it was a selection field
+    # remove the existing linked `ir_model_fields_selection` records in case it was a selection field
     if table_exists(cr, "ir_model_fields_selection"):
         cr.execute(
             """
@@ -1008,6 +1008,9 @@ def __init__(self, list_=(), quoted=()):
         self._trailing_comma = False
         self._alias = None
 
+    def __hash__(self):
+        return hash((tuple(self._unquoted_columns), self._leading_comma, self._trailing_comma, self._alias))
+
     def __eq__(self, other):
         return (
             isinstance(other, ColumnList)

src/util/records.py

@@ -1077,7 +1077,7 @@ def __update_record_from_xml(
     fields,
     done_refs,
 ):
-    from .modules import get_manifest
+    from .modules import get_manifest  # noqa: PLC0415
 
     if "." not in xmlid:
         raise ValueError("Please use fully qualified name <module>.<name>")

src/util/report.py

@@ -8,6 +8,11 @@
 import lxml
 from docutils.core import publish_string
 
+try:
+    import markdown
+except ImportError:
+    markdown = None
+
 from .helpers import _validate_model
 from .misc import parse_version
 
@@ -189,7 +194,7 @@ def rst2html(rst):
 
 
 def md2html(md):
-    import markdown
+    assert markdown
 
     mdversion = markdown.__version_info__ if hasattr(markdown, "__version_info__") else markdown.version_info
     extensions = [
@@ -259,9 +264,9 @@ def ref(xid):
 
     except MigrationError:
         try:
-            from openerp.modules.registry import RegistryManager
+            from openerp.modules.registry import RegistryManager  # noqa: PLC0415
         except ImportError:
-            from openerp.modules.registry import Registry as RegistryManager
+            from openerp.modules.registry import Registry as RegistryManager  # noqa: PLC0415
         registry = RegistryManager.get(cr.dbname)
         user = registry["res.users"].browse(cr, SUPERUSER_ID, uid, context=ctx)