Merge PR #71 into master

Signed-off-by legalsylvain

Author: OCA-git-bot

.github/workflows/pre-commit.yml

@@ -0,0 +1,36 @@
+name: pre-commit
+
+on:
+  pull_request:
+    branches:
+      - "*"
+  push:
+    branches:
+      - "master"
+
+jobs:
+  pre-commit:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v2
+      - uses: actions/setup-python@v2
+        with:
+          python-version: "3.8"
+      - name: Get python version
+        run: echo "PY=$(python -VV | sha256sum | cut -d' ' -f1)" >> $GITHUB_ENV
+      - uses: actions/cache@v1
+        with:
+          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

@@ -0,0 +1,16 @@
+exclude: |
+  (?x)
+  tests/data_template/|
+  tests/data_result/
+
+repos:
+-   repo: https://github.com/pre-commit/pre-commit-hooks
+    rev: v2.3.0
+    hooks:
+    -   id: check-yaml
+    -   id: end-of-file-fixer
+    -   id: trailing-whitespace
+-   repo: https://github.com/psf/black
+    rev: 22.10.0
+    hooks:
+    -   id: black

.travis.yml

@@ -19,7 +19,6 @@ install:
 # command to run tests
 script:
   - git --version
-  - flake8 . --exclude=__init__.py,tests/data_*
   - coverage run --source odoo_module_migrate setup.py test
 
 after_success:

MANIFEST.in

@@ -1,2 +1,2 @@
 include requirements.txt
-recursive-include odoo_module_migrate/migration_scripts/ *.py *.yaml
+recursive-include odoo_module_migrate/migration_scripts/ *.py *.yaml

README.rst

@@ -3,18 +3,16 @@
     :alt: License: AGPL-3
 .. image:: https://img.shields.io/badge/python-3.6-blue.svg
     :alt: Python support: 3.6
-.. image:: https://travis-ci.org/grap/odoo-module-migrator.svg?branch=master
-    :target: https://travis-ci.org/grap/odoo-module-migrator
-.. image:: https://coveralls.io/repos/grap/odoo-module-migrator/badge.png?branch=master
-    :target: https://coveralls.io/r/grap/odoo-module-migrator?branch=master
+.. image:: https://app.travis-ci.com/OCA/odoo-module-migrator.svg?branch=master
+    :target: https://app.travis-ci.com/OCA/odoo-module-migrator
 
 ====================
 odoo-module-migrator
 ====================
 
-``odoo-module-migrator`` is a python3 library that allows you to automatically migrate 
+``odoo-module-migrator`` is a python3 library that allows you to automatically migrate
 module code to make it compatible with newer Odoo version.
-for exemple: 
+for exemple:
 
 * renaming ``__openerp__.py`` file into ``__manifest__.py``
 * removing ``# -*- encoding: utf-8 -*-`` since V11.0
@@ -243,4 +241,3 @@ Contributors
 ------------
 
 * Sylvain LE GAL (https://www.twitter.com/legalsylvain)
-

odoo_module_migrate/__main__.py

@@ -13,9 +13,7 @@
 
 def get_parser():
 
-    main_parser = argparse.ArgumentParser(
-        formatter_class=argparse.RawTextHelpFormatter
-    )
+    main_parser = argparse.ArgumentParser(formatter_class=argparse.RawTextHelpFormatter)
 
     main_parser.add_argument(
         "-d",
@@ -62,16 +60,16 @@ def get_parser():
     main_parser.add_argument(
         "-fp",
         "--format-patch",
-        action='store_true',
+        action="store_true",
         help="Enable this option, if you want to get the code from the"
-        " previous branch."
+        " previous branch.",
     )
 
     main_parser.add_argument(
         "-rn",
         "--remote-name",
         dest="remote_name",
-        default='origin',
+        default="origin",
         type=str,
     )
 
@@ -95,22 +93,27 @@ def get_parser():
     main_parser.add_argument(
         "-nc",
         "--no-commit",
-        action='store_true',
+        action="store_true",
         default=False,
         help="Enable this option, if you don't want that the library commits"
-        " the changes. (using git add and git commit command)"
+        " the changes. (using git add and git commit command)",
     )
 
     # TODO: Move to `argparse.BooleanOptionalAction` once in Python 3.9+
     main_parser.add_argument(
-        "-npc", "--no-pre-commit", dest="pre_commit", action="store_false",
+        "-npc",
+        "--no-pre-commit",
+        dest="pre_commit",
+        action="store_false",
         help="Skip pre-commit execution",
     )
 
     # TODO: Move to `argparse.BooleanOptionalAction` once in Python 3.9+
     main_parser.add_argument(
-        "-nrmf", "--no-remove-migration-folder",
-        dest="remove_migration_folder", action="store_false",
+        "-nrmf",
+        "--no-remove-migration-folder",
+        dest="remove_migration_folder",
+        action="store_false",
         help="Skip removing migration folder",
     )
 
@@ -131,13 +134,22 @@ def main(args=False):
 
     try:
         # Create a new Migration Object
-        module_names = args.modules\
-            and [x.strip() for x in args.modules.split(",") if x.strip()] or []
+        module_names = (
+            args.modules
+            and [x.strip() for x in args.modules.split(",") if x.strip()]
+            or []
+        )
 
         migration = Migration(
-            args.directory, args.init_version_name, args.target_version_name,
-            module_names, args.format_patch, args.remote_name,
-            not args.no_commit, args.pre_commit, args.remove_migration_folder,
+            args.directory,
+            args.init_version_name,
+            args.target_version_name,
+            module_names,
+            args.format_patch,
+            args.remote_name,
+            not args.no_commit,
+            args.pre_commit,
+            args.remove_migration_folder,
         )
 
         # run Migration