[IMP] update dotfiles

.copier-answers.yml

@@ -1,14 +1,17 @@
 # Do NOT update manually; changes here will be overwritten by Copier
-_commit: v1.3.6
+_commit: v1.5.2
 _src_path: gh:oca/oca-addons-repo-template
 ci: GitHub
 dependency_installation_mode: PIP
 generate_requirements_txt: true
 include_wkhtmltopdf: false
 odoo_version: 14.0
+org_name: Odoo Community Association (OCA)
+org_slug: OCA
 rebel_module_groups: []
 repo_description: Addons concerning Odoo Italian Localization.
 repo_name: l10n-italy
 repo_slug: l10n-italy
+repo_website: https://github.com/OCA/l10n-italy
 travis_apt_packages: []
 travis_apt_sources: []

.eslintrc.yml

@@ -4,7 +4,7 @@ env:
 
 # See https://github.com/OCA/odoo-community.org/issues/37#issuecomment-470686449
 parserOptions:
-  ecmaVersion: 2017
+  ecmaVersion: 2019
 
 overrides:
   - files:

.github/workflows/pre-commit.yml

@@ -2,19 +2,34 @@ name: pre-commit
 
 on:
   pull_request:
+    branches:
+      - "14.0*"
   push:
+    branches:
+      - "14.0"
+      - "14.0-ocabot-*"
 
 jobs:
   pre-commit:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v2
       - uses: actions/setup-python@v2
+      - name: Get python version
+        run: echo "PY=$(python -VV | sha256sum | cut -d' ' -f1)" >> $GITHUB_ENV
+      - uses: actions/cache@v1
         with:
-          # The pylint-odoo version we use here does not support python 3.10
-          # https://github.com/OCA/oca-addons-repo-template/issues/80
-          # We also need to pin to an older version of python for older odoo versions
-          # where we are not using black > 21. Older black versions won't work with
-          # Python 3.9.8+, and we can't bump black without reformatting.
-          python-version: "3.9.7"
-      - uses: pre-commit/[email protected]
+          path: ~/.cache/pre-commit
+          key: pre-commit|${{ env.PY }}|${{ hashFiles('.pre-commit-config.yaml') }}
+      - name: Install pre-commit
+        run: pip install pre-commit
+      - name: Run pre-commit
+        run: pre-commit run --all-files --show-diff-on-failure --color=always
+      - name: Check that all files generated by pre-commit are in git
+        run: |
+          newfiles="$(git ls-files --others --exclude-from=.gitignore)"
+          if [ "$newfiles" != "" ] ; then
+              echo "Please check-in the following files:"
+              echo "$newfiles"
+              exit 1
+          fi

.github/workflows/test.yml

@@ -1,24 +1,45 @@
-name: test Odoo addons
+name: tests
 
 on:
   pull_request:
     branches:
       - "14.0*"
   push:
     branches:
-      - "14.0*"
+      - "14.0"
+      - "14.0-ocabot-*"
 
 jobs:
+  unreleased-deps:
+    runs-on: ubuntu-latest
+    name: Detect unreleased dependencies
+    steps:
+      - uses: actions/checkout@v2
+      - run: |
+          for reqfile in requirements.txt test-requirements.txt ; do
+              if [ -f ${reqfile} ] ; then
+                  result=0
+                  # reject non-comment lines that contain a / (i.e. URLs, relative paths)
+                  grep "^[^#].*/" ${reqfile} || result=$?
+                  if [ $result -eq 0 ] ; then
+                      echo "Unreleased dependencies found in ${reqfile}."
+                      exit 1
+                  fi
+              fi
+          done
   test:
     runs-on: ubuntu-latest
     container: ${{ matrix.container }}
+    name: ${{ matrix.name }}
     strategy:
       fail-fast: false
       matrix:
         include:
           - container: ghcr.io/oca/oca-ci/py3.6-odoo14.0:latest
             makepot: "true"
+            name: test with Odoo
           - container: ghcr.io/oca/oca-ci/py3.6-ocb14.0:latest
+            name: test with OCB
     services:
       postgres:
         image: postgres:9.6

.pre-commit-config.yaml

@@ -56,7 +56,7 @@ repos:
           - --remove-duplicate-keys
           - --remove-unused-variables
   - repo: https://github.com/psf/black
-    rev: 20.8b1
+    rev: 22.3.0
     hooks:
       - id: black
   - repo: https://github.com/pre-commit/mirrors-prettier
@@ -130,7 +130,7 @@ repos:
         name: flake8
         additional_dependencies: ["flake8-bugbear==20.1.4"]
   - repo: https://github.com/PyCQA/pylint
-    rev: pylint-2.5.3
+    rev: v2.11.1
     hooks:
       - id: pylint
         name: pylint with optional checks
@@ -140,7 +140,7 @@ repos:
           - --ignore=l10n_it_fatturapa_in/tests/data/
         verbose: true
         additional_dependencies: &pylint_deps
-          - pylint-odoo==3.5.0
+          - pylint-odoo==5.0.5
       - id: pylint
         name: pylint with mandatory checks
         args:

README.md

@@ -1,5 +1,7 @@
-[![Runbot Status](https://runbot.odoo-community.org/runbot/badge/flat/122/14.0.svg)](https://runbot.odoo-community.org/runbot/repo/github-com-oca-l10n-italy-122)
-[![Build Status](https://travis-ci.com/OCA/l10n-italy.svg?branch=14.0)](https://travis-ci.com/OCA/l10n-italy)
+
+[![Runboat](https://img.shields.io/badge/runboat-Try%20me-875A7B.png)](https://runboat.odoo-community.org/builds?repo=OCA/l10n-italy&target_branch=14.0)
+[![Pre-commit Status](https://github.com/OCA/l10n-italy/actions/workflows/pre-commit.yml/badge.svg?branch=14.0)](https://github.com/OCA/l10n-italy/actions/workflows/pre-commit.yml?query=branch%3A14.0)
+[![Build Status](https://github.com/OCA/l10n-italy/actions/workflows/test.yml/badge.svg?branch=14.0)](https://github.com/OCA/l10n-italy/actions/workflows/test.yml?query=branch%3A14.0)
 [![codecov](https://codecov.io/gh/OCA/l10n-italy/branch/14.0/graph/badge.svg)](https://codecov.io/gh/OCA/l10n-italy)
 [![Translation Status](https://translation.odoo-community.org/widgets/l10n-italy-14-0/-/svg-badge.svg)](https://translation.odoo-community.org/engage/l10n-italy-14-0/?utm_source=widget)
 
@@ -81,12 +83,11 @@ addon | version | maintainers | summary
 
 This repository is licensed under [AGPL-3.0](LICENSE).
 
-However, each module can have a totally different license, as long as they adhere to OCA
+However, each module can have a totally different license, as long as they adhere to Odoo Community Association (OCA)
 policy. Consult each module's `__manifest__.py` file, which contains a `license` key
 that explains its license.
 
 ----
-
 OCA, or the [Odoo Community Association](http://odoo-community.org/), is a nonprofit
 organization whose mission is to support the collaborative development of Odoo features
 and promote its widespread use.

l10n_it_account/models/account_group.py

@@ -73,7 +73,7 @@ def get_account_balance_sign(self):
         return 1
 
     def get_group_accounts(self):
-        """ Retrieves every account from `self` and `self`'s subgroups. """
+        """Retrieves every account from `self` and `self`'s subgroups."""
         return (self + self.get_group_subgroups()).mapped("account_ids")
 
     def get_group_progenitor(self):
@@ -97,6 +97,6 @@ def get_group_parents(self):
         return self.browse(parent_ids)
 
     def get_group_subgroups(self):
-        """ Retrieves every subgroup for groups `self`. """
+        """Retrieves every subgroup for groups `self`."""
         subgroups_ids = self.search([("id", "child_of", self.ids)])
         return subgroups_ids

l10n_it_account_tax_kind/tests/test_account_tax_kind.py

@@ -12,7 +12,7 @@ def test_compute_display_name(self):
         self.tax_kind_n1._compute_display_name()
         self.assertEqual(
             self.tax_kind_n1.display_name,
-            u"[{}] {}".format(self.tax_kind_n1.code, self.tax_kind_n1.name),
+            "[{}] {}".format(self.tax_kind_n1.code, self.tax_kind_n1.name),
         )
 
     def test_name_search(self):

l10n_it_fatturapa/models/account.py

@@ -88,7 +88,7 @@ class WelfareFundType(models.Model):
     def name_get(self):
         res = []
         for record in self:
-            res.append((record.id, u"[{}] {}".format(record.name, record.description)))
+            res.append((record.id, "[{}] {}".format(record.name, record.description)))
         return res
 
 

l10n_it_fatturapa/models/ir_attachment.py

@@ -19,7 +19,7 @@
     _logger.debug(err)
 
 
-re_base64 = re.compile(br"^([A-Za-z0-9+/]{4})*([A-Za-z0-9+/]{3}=|[A-Za-z0-9+/]{2}==)?$")
+re_base64 = re.compile(rb"^([A-Za-z0-9+/]{4})*([A-Za-z0-9+/]{3}=|[A-Za-z0-9+/]{2}==)?$")
 
 
 def is_base64(s):

l10n_it_fatturapa_in/models/attachment.py

@@ -74,7 +74,7 @@ def _compute_e_invoice_validation_error(self):
             if not bills_with_error:
                 continue
             att.e_invoice_validation_error = True
-            errors_message_template = u"{bill}:\n{errors}"
+            errors_message_template = "{bill}:\n{errors}"
             error_messages = list()
             for bill in bills_with_error:
                 error_messages.append(

l10n_it_fatturapa_in/tests/test_import_fatturapa_xml.py

@@ -144,8 +144,8 @@ def test_04_xml_import(self):
                 self.assertEqual(e_line.cod_article_ids[0].code_val, "12345")
         self.assertEqual(
             invoice.inconsistencies,
-            u"Company Name field contains 'Societa' "
-            u"Alpha SRL'. Your System contains 'SOCIETA' ALPHA SRL'\n\n",
+            "Company Name field contains 'Societa' "
+            "Alpha SRL'. Your System contains 'SOCIETA' ALPHA SRL'\n\n",
         )
         # allow following test to reuse the same XML file
         invoice.ref = invoice.payment_reference = "14041"
@@ -344,13 +344,13 @@ def test_13_xml_import(self):
         invoice2 = self.invoice_model.browse(invoice2_id)
         self.assertEqual(
             invoice1.inconsistencies,
-            u"Company Name field contains 'Societa' "
-            u"Alpha SRL'. Your System contains 'SOCIETA' ALPHA SRL'\n\n",
+            "Company Name field contains 'Societa' "
+            "Alpha SRL'. Your System contains 'SOCIETA' ALPHA SRL'\n\n",
         )
         self.assertEqual(
             invoice2.inconsistencies,
-            u"Company Name field contains 'Societa' "
-            u"Alpha SRL'. Your System contains 'SOCIETA' ALPHA SRL'\n\n",
+            "Company Name field contains 'Societa' "
+            "Alpha SRL'. Your System contains 'SOCIETA' ALPHA SRL'\n\n",
         )
 
     def test_14_xml_import(self):
@@ -366,9 +366,9 @@ def test_14_xml_import(self):
         self.assertEqual(invoice.amount_tax, 0.0)
         self.assertEqual(
             invoice.inconsistencies,
-            u"Company Name field contains 'Societa' "
+            "Company Name field contains 'Societa' "
             "Alpha SRL'. Your System contains 'SOCIETA' ALPHA SRL'\n\n"
-            u"XML contains tax with percentage '15.55'"
+            "XML contains tax with percentage '15.55'"
             " but it does not exist in your system\n"
             "XML contains tax with percentage '15.55'"
             " but it does not exist in your system",

l10n_it_fatturapa_out_rc/wizard/efattura.py

@@ -10,14 +10,10 @@ class EFatturaOut(_EFatturaOut):
     def get_template_values(self):
         def get_sign(invoice):
             sign = 1
-            if (
-                invoice.move_type
-                in [
-                    "out_refund",
-                    "in_refund",
-                ]
-                and invoice.fiscal_document_type_id.code not in ["TD04", "TD08"]
-            ):
+            if invoice.move_type in [
+                "out_refund",
+                "in_refund",
+            ] and invoice.fiscal_document_type_id.code not in ["TD04", "TD08"]:
                 sign = -1
             return sign