Merge PR #482 into 17.0

Signed-off-by pedrobaeza

.copier-answers.yml

@@ -1,5 +1,5 @@
 # Do NOT update manually; changes here will be overwritten by Copier
-_commit: v1.19
+_commit: v1.21.1
 _src_path: gh:oca/oca-addons-repo-template
 additional_ruff_rules: []
 ci: GitHub

.github/workflows/stale.yml

@@ -9,7 +9,7 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: Stale PRs and issues policy
-        uses: actions/stale@v4
+        uses: actions/stale@v9
         with:
           repo-token: ${{ secrets.GITHUB_TOKEN }}
           # General settings.
@@ -48,7 +48,7 @@ jobs:
       # * Issues that are pending more information
       # * Except Issues marked as "no stale"
       - name: Needs more information stale issues policy
-        uses: actions/stale@v4
+        uses: actions/stale@v9
         with:
           repo-token: ${{ secrets.GITHUB_TOKEN }}
           ascending: true

.github/workflows/test.yml

@@ -63,7 +63,9 @@ jobs:
         run: oca_init_test_database
       - name: Run tests
         run: oca_run_tests
-      - uses: codecov/codecov-action@v1
+      - uses: codecov/codecov-action@v4
+        with:
+          token: ${{ secrets.CODECOV_TOKEN }}
       - name: Update .pot files
         run: oca_export_and_push_pot https://x-access-token:${{ secrets.GIT_PUSH_TOKEN }}@github.com/${{ github.repository }}
         if: ${{ matrix.makepot == 'true' && github.event_name == 'push' && github.repository_owner == 'OCA' }}

.pre-commit-config.yaml

@@ -16,6 +16,8 @@ exclude: |
   readme/.*\.(rst|md)$|
   # Ignore build and dist directories in addons
   /build/|/dist/|
+  # Ignore test files in addons
+  /tests/samples/.*|
   # You don't usually want a bot to modify your legal texts
   (LICENSE.*|COPYING.*)
 default_language_version:
@@ -41,7 +43,7 @@ repos:
     hooks:
       - id: whool-init
   - repo: https://github.com/oca/maintainer-tools
-    rev: f71041f22b8cd68cf7c77b73a14ca8d8cd190a60
+    rev: 9a170331575a265c092ee6b24b845ec508e8ef75
     hooks:
       # update the NOT INSTALLABLE ADDONS section above
       - id: oca-update-pre-commit-excluded-addons
@@ -110,7 +112,7 @@ repos:
         args: [--fix, --exit-non-zero-on-fix]
       - id: ruff-format
   - repo: https://github.com/OCA/pylint-odoo
-    rev: v8.0.19
+    rev: v9.0.4
     hooks:
       - id: pylint_odoo
         name: pylint with optional checks

.ruff.toml

@@ -6,6 +6,7 @@ fix = true
 extend-select = [
     "B",
     "C90",
+    "E501",  # line too long (default 88)
     "I",  # isort
     "UP",  # pyupgrade
 ]

connector/components/binder.py

@@ -141,6 +141,7 @@ def unwrap_model(self):
             column = self.model._fields[self._odoo_field]
         except KeyError as err:
             raise ValueError(
-                f"Cannot unwrap model {self.model._name}, because it has no {self._odoo_field} fields"
+                f"Cannot unwrap model {self.model._name}, because it has no "
+                f"{self._odoo_field} fields"
             ) from err
         return column.comodel_name

connector/components/locker.py

@@ -63,7 +63,8 @@ def lock(self, records, seconds=None, ignore_retry=True):
             )
             raise RetryableJobError(
                 "A concurrent job is already working on the same record "
-                f"({self.model._name} with one id in {tuple(records.ids)}). The job will be retried later.",
+                f"({self.model._name} with one id in {tuple(records.ids)}). The job "
+                f"will be retried later.",
                 seconds=seconds,
                 ignore_retry=ignore_retry,
             ) from err

connector/components/synchronizer.py

@@ -230,7 +230,8 @@ def _lock(self):
             )
             raise RetryableJobError(
                 "A concurrent job is already exporting the same record "
-                f"({self.model._name} with id {self.binding.id}). The job will be retried later."
+                f"({self.model._name} with id {self.binding.id}). The job will be "
+                f"retried later."
             ) from err
 
     def _has_to_skip(self):

connector/tests/test_mapper.py

@@ -145,7 +145,7 @@ def name(self):
                 pass
 
         class FryMapperInherit(Component):
-            _inherit = "fry.mapper"
+            _inherit = "fry.mapper"  # pylint: disable=R8180
             _apply_on = "res.users"
 
             @changed_by("email")