[IMP] update dotfiles [ci skip]

Author: OCA-git-bot

.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: Travis
 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: 'TODO: add repo description.'
 repo_name: delivery-carrier
 repo_slug: delivery-carrier
+repo_website: https://github.com/OCA/delivery-carrier
 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/action@v2.0.0
+          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

.pre-commit-config.yaml

@@ -46,7 +46,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
@@ -120,7 +120,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
@@ -129,7 +129,7 @@ repos:
           - --exit-zero
         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,4 +1,5 @@
-[![Runbot Status](https://runbot.odoo-community.org/runbot/badge/flat/99/14.0.svg)](https://runbot.odoo-community.org/runbot/repo/github-com-oca-delivery-carrier-99)
+
+[![Runboat](https://img.shields.io/badge/runboat-Try%20me-875A7B.png)](https://runboat.odoo-community.org/builds?repo=OCA/delivery-carrier&target_branch=14.0)
 [![Build Status](https://travis-ci.com/OCA/delivery-carrier.svg?branch=14.0)](https://travis-ci.com/OCA/delivery-carrier)
 [![codecov](https://codecov.io/gh/OCA/delivery-carrier/branch/14.0/graph/badge.svg)](https://codecov.io/gh/OCA/delivery-carrier)
 [![Translation Status](https://translation.odoo-community.org/widgets/delivery-carrier-14-0/-/svg-badge.svg)](https://translation.odoo-community.org/engage/delivery-carrier-14-0/?utm_source=widget)
@@ -58,12 +59,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.

base_delivery_carrier_files/tests/test_base_delivery_carrier_files.py

@@ -73,7 +73,7 @@ def setUp(self):
         self.picking_type = self.env.ref("stock.warehouse0").out_type_id
 
     def test_carrier_file_generation(self):
-        """ Test carrier file generation """
+        """Test carrier file generation"""
         # I confirm outgoing shipment of 130 kgm Ice-cream.
         picking = self.env["stock.picking"].create(
             {
@@ -118,7 +118,7 @@ def test_carrier_file_generation(self):
         self.assertFalse(new_picking.carrier_file_generated)
 
     def test_manual_carrier_file_generation(self):
-        """ Test manual carrier file generation """
+        """Test manual carrier file generation"""
         # I confirm outgoing shipment of 130 kgm Ice-cream.
         picking = self.env["stock.picking"].create(
             {

base_delivery_carrier_label/models/carrier_account.py

@@ -12,7 +12,7 @@ class CarrierAccount(models.Model):
 
     @api.model
     def _selection_file_format(self):
-        """ To inherit to add label file types"""
+        """To inherit to add label file types"""
         return [("PDF", "PDF"), ("ZPL", "ZPL"), ("XML", "XML")]
 
     name = fields.Char(required=True)

base_delivery_carrier_label/models/delivery_carrier.py

@@ -19,7 +19,7 @@ def alternative_send_shipping(self, pickings):
         return {}
 
     def default_options(self):
-        """ Returns default and available options for a carrier """
+        """Returns default and available options for a carrier"""
         options = self.env["delivery.carrier.option"].browse()
         for available_option in self.available_option_ids:
             if available_option.mandatory or available_option.by_default:

base_delivery_carrier_label/models/delivery_carrier_template_option.py

@@ -6,7 +6,7 @@
 
 
 class DeliveryCarrierTemplateOption(models.Model):
-    """ Available options for a carrier (partner) """
+    """Available options for a carrier (partner)"""
 
     _name = "delivery.carrier.template.option"
     _description = "Delivery carrier template option"

base_delivery_carrier_label/models/shipping_label.py

@@ -6,7 +6,7 @@
 
 
 class ShippingLabel(models.Model):
-    """ Child class of ir attachment to identify which are labels """
+    """Child class of ir attachment to identify which are labels"""
 
     _name = "shipping.label"
     _inherits = {"ir.attachment": "attachment_id"}

base_delivery_carrier_label/models/stock_picking.py

@@ -74,7 +74,7 @@ def send_to_shipper(self):
 
     @api.onchange("carrier_id")
     def onchange_carrier_id(self):
-        """ Inherit this method in your module """
+        """Inherit this method in your module"""
         if not self.carrier_id:
             return
         # This can look useless as the field carrier_code and
@@ -159,7 +159,7 @@ def _get_carrier_account_domain(self):
         ]
 
     def _get_carrier_account(self):
-        """ Return a carrier suitable for the current picking """
+        """Return a carrier suitable for the current picking"""
         domain = self._get_carrier_account_domain()
         return self.env["carrier.account"].search(
             domain,
@@ -191,7 +191,7 @@ def _get_label_sender_address(self):
         return self.env["res.partner"].browse(address_id)
 
     def _check_existing_shipping_label(self):
-        """ Check that labels don't already exist for this picking """
+        """Check that labels don't already exist for this picking"""
         self.ensure_one()
         labels = self.env["shipping.label"].search(
             [("res_id", "=", self.id), ("res_model", "=", "stock.picking")]

base_delivery_carrier_label/models/stock_quant_package.py

@@ -47,5 +47,5 @@ def _complete_name(self, name, args):
         return res
 
     def open_website_url(self):
-        """ Implement you own action in your module """
+        """Implement you own action in your module"""
         return None

delivery_carrier_label_batch/models/stock_batch_picking.py

@@ -38,7 +38,7 @@ def _get_options_to_add(self, carrier=None):
 
     @api.onchange("carrier_id")
     def carrier_id_change(self):
-        """ Inherit this method in your module """
+        """Inherit this method in your module"""
         if self.carrier_id:
             # This can look useless as the field carrier_code and
             # carrier_type are related field. But it's needed to fill

delivery_carrier_label_batch/tests/test_generate_labels.py

@@ -9,7 +9,7 @@
 
 class TestGenerateLabels(common.SavepointCase):
 
-    """ Test the wizard for delivery carrier label generation """
+    """Test the wizard for delivery carrier label generation"""
 
     @classmethod
     def setUpClass(cls):

delivery_postlogistics/models/delivery_carrier.py

@@ -8,7 +8,7 @@
 
 
 class DeliveryCarrier(models.Model):
-    """ Add service group """
+    """Add service group"""
 
     _inherit = "delivery.carrier"
 

delivery_postlogistics/models/postlogistics_carrier_template_options.py

@@ -15,7 +15,7 @@
 
 
 class DeliveryCarrierTemplateOption(models.Model):
-    """ Available options for a carrier (partner) """
+    """Available options for a carrier (partner)"""
 
     _name = "postlogistics.delivery.carrier.template.option"
     _description = "Delivery carrier template option"

delivery_postlogistics/models/postlogistics_shipping_label.py

@@ -5,7 +5,7 @@
 
 
 class PostlogisticsShippingLabel(models.Model):
-    """ Child class of ir attachment to identify which are labels """
+    """Child class of ir attachment to identify which are labels"""
 
     _name = "postlogistics.shipping.label"
     _inherits = {"ir.attachment": "attachment_id"}

delivery_postlogistics/models/stock_picking.py

@@ -27,7 +27,7 @@ class StockPicking(models.Model):
     )
 
     def _get_packages_from_picking(self):
-        """ Get all the packages from the picking """
+        """Get all the packages from the picking"""
         self.ensure_one()
         operation_obj = self.env["stock.move.line"]
         operations = operation_obj.search(
@@ -184,7 +184,7 @@ def write_tracking_number_label(self, label_result, packages):
     def _generate_postlogistics_label(
         self, webservice_class=None, package_ids=None, skip_attach_file=False
     ):
-        """ Generate labels and write tracking numbers received """
+        """Generate labels and write tracking numbers received"""
         self.ensure_one()
         user = self.env.user
         company = user.company_id
@@ -231,6 +231,6 @@ def _generate_postlogistics_label(
         return labels
 
     def generate_postlogistics_shipping_labels(self, package_ids=None):
-        """ Add label generation for PostLogistics """
+        """Add label generation for PostLogistics"""
         self.ensure_one()
         return self._generate_postlogistics_label(package_ids=package_ids)

delivery_postlogistics/postlogistics/web_service.py

@@ -297,7 +297,7 @@ def _get_item_number(self, picking, pack_num):
         return "%02d%s" % (pack_num, picking_num[-6:].zfill(6))
 
     def _prepare_item_list(self, picking, recipient, packages):
-        """ Return a list of item made from the pickings """
+        """Return a list of item made from the pickings"""
         carrier = picking.carrier_id
         item_list = []
         pack_counter = 1

delivery_roulier/models/stock_picking.py

@@ -196,7 +196,7 @@ def _roulier_convert_address(self, partner):
 
         for tel in ["mobile", "phone"]:
             if address.get(tel):
-                address[tel] = address[tel].replace(u"\u00A0", "").replace(" ", "")
+                address[tel] = address[tel].replace("\u00A0", "").replace(" ", "")
 
         address["phone"] = address.get("mobile", address.get("phone"))