[REM] hr-expense: remove _do_create_moves from hr_expense_sheet inheritance.

  - the _do_create_moves from hr_expense_sheet inheritance is not longer
  required, it has been removed.

Author: Hernannc00000001

Diff for: .gitignore

@@ -82,3 +82,19 @@ docs/_build/
 
 # OCA rules
 !static/lib/
+
+# pre-commit-vauxoo
+.bandit*
+.config/
+.editorconfig
+.eslintrc*
+.flake8*
+.isort.cfg
+.oca_hooks*
+.pre-commit*.yaml
+.prettierrc.yml
+.pylintrc*
+bandit*.yaml
+doc8.ini
+pyproject.toml
+

Diff for: .pre-commit-config.yaml

@@ -1,100 +1,74 @@
 exclude: |
-  (?x)
-  # NOT INSTALLABLE ADDONS
-  # END NOT INSTALLABLE ADDONS
-  # Files and folders generated by bots, to avoid loops
-  ^setup/|/static/description/index\.html$|
-  # We don't want to mess with tool-generated files
-  .svg$|/tests/([^/]+/)?cassettes/|^.copier-answers.yml$|^.github/|
-  # Maybe reactivate this when all README files include prettier ignore tags?
-  ^README\.md$|
-  # Library files can have extraneous formatting (even minimized)
-  /static/(src/)?lib/|
-  # Repos using Sphinx to generate docs don't need prettying
-  ^docs/_templates/.*\.html$|
-  # Don't bother non-technical authors with formatting issues in docs
-  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.*)
+  (?x)(
+    # NOT INSTALLABLE ADDONS
+    # END NOT INSTALLABLE ADDONS
+    # Files and folders generated by bots, to avoid loops
+    ^setup/|/static/description/index\.html$|
+    # We don't want to mess with tool-generated files
+    .svg$|/tests/([^/]+/)?cassettes/|
+    # Maybe reactivate this when all README files include prettier ignore tags?
+    ^README\.md$|
+    # Library files can have extraneous formatting (even minimized)
+    /static/(src/)?lib/|
+    # Repos using Sphinx to generate docs don't need prettying
+    ^docs/_templates/.*\.html$|
+    # You don't usually want a bot to modify your legal texts
+    (LICENSE.*|COPYING.*)|
+    (\.travis\.yml|\.gitlab\-ci\.yml|\.pre\-commit\-config*\.yaml)|
+    # It was ignored from original MQT since that there is not init file
+    (/migrations/1[0-4])|
+    (/doc/|/docs/)|
+    # Legacy modules absa: galaxy
+    (galaxy/)|
+    # External scripts
+    (scripts/)
+  )
 default_language_version:
   python: python3
-  node: "16.17.0"
+  node: "14.13.0"
 repos:
-  - repo: local
-    hooks:
-      # These files are most likely copier diff rejection junks; if found,
-      # review them manually, fix the problem (if needed) and remove them
-      - id: forbidden-files
-        name: forbidden files
-        entry: found forbidden files; remove them
-        language: fail
-        files: "\\.rej$"
-      - id: en-po-files
-        name: en.po files cannot exist
-        entry: found a en.po file
-        language: fail
-        files: '[a-zA-Z0-9_]*/i18n/en\.po$'
-  - repo: https://github.com/sbidoul/whool
-    rev: v0.5
-    hooks:
-      - id: whool-init
-  - repo: https://github.com/oca/maintainer-tools
-    rev: d5fab7ee87fceee858a3d01048c78a548974d935
+  - repo: https://github.com/PyCQA/flake8
+    rev: 6.1.0
     hooks:
-      # update the NOT INSTALLABLE ADDONS section above
-      - id: oca-update-pre-commit-excluded-addons
-      - id: oca-fix-manifest-website
-        args: ["https://github.com/OCA/hr-expense"]
-      - id: oca-gen-addon-readme
-        args:
-          - --addons-dir=.
-          - --branch=17.0
-          - --org-name=OCA
-          - --repo-name=hr-expense
-          - --if-source-changed
-          - --keep-source-digest
-          - --convert-fragments-to-markdown
-      - id: oca-gen-external-dependencies
-  - repo: https://github.com/OCA/odoo-pre-commit-hooks
-    rev: v0.0.25
-    hooks:
-      - id: oca-checks-odoo-module
-      - id: oca-checks-po
-  - repo: https://github.com/pre-commit/mirrors-prettier
-    rev: v2.7.1
+      - id: flake8
+        name: flake8 mandatory checks
+  - repo: https://github.com/OCA/pylint-odoo
+    rev: v9.0.5
     hooks:
-      - id: prettier
-        name: prettier (with plugin-xml)
-        additional_dependencies:
-          - "[email protected]"
-          - "@prettier/[email protected]"
+      - id: pylint_odoo
+        name: pylint mandatory checks
         args:
-          - --plugin=@prettier/plugin-xml
-        files: \.(css|htm|html|js|json|jsx|less|md|scss|toml|ts|xml|yaml|yml)$
+          # External scripts
+          - --disable=R0000
   - repo: https://github.com/pre-commit/mirrors-eslint
-    rev: v8.24.0
+    rev: v8.51.0
     hooks:
       - id: eslint
-        verbose: true
+        name: javascript mandatory checks
         args:
           - --color
-          - --fix
+          - --config=.eslintrc.json
+          # - --fix
+  - repo: local
+    hooks:
+      - id: name-non-ascii
+        name: Use only ASCII (no accents or special characters) for filenames and directory names
+        description: It may raise errors on OS without locale settings configured
+        entry: Use only ASCII (no accents or special characters) for filenames and directory names
+        language: fail
+        files: '[^\x00-\x7F]'
+      - id: vx-check-deactivate
+        name: deactivate.jinja syntax check.
+        description: Validate deactivate.jinja files' syntax.
+        entry: vx-check-deactivate
+        args: []
+        language: python
+        types: [jinja]
+        files: ^deactivate\.jinja$
   - repo: https://github.com/pre-commit/pre-commit-hooks
-    rev: v4.3.0
+    rev: v4.5.0
     hooks:
-      - id: trailing-whitespace
-        # exclude autogenerated files
-        exclude: /README\.rst$|\.pot?$
-      - id: end-of-file-fixer
-        # exclude autogenerated files
-        exclude: /README\.rst$|\.pot?$
       - id: debug-statements
-      - id: fix-encoding-pragma
-        args: ["--remove"]
       - id: check-case-conflict
       - id: check-docstring-first
       - id: check-executables-have-shebangs
@@ -103,23 +77,3 @@ repos:
         exclude: /README\.rst$|^docs/.*\.rst$
       - id: check-symlinks
       - id: check-xml
-      - id: mixed-line-ending
-        args: ["--fix=lf"]
-  - repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: v0.1.3
-    hooks:
-      - id: ruff
-        args: [--fix, --exit-non-zero-on-fix]
-      - id: ruff-format
-  - repo: https://github.com/OCA/pylint-odoo
-    rev: v9.0.4
-    hooks:
-      - id: pylint_odoo
-        name: pylint with optional checks
-        args:
-          - --rcfile=.pylintrc
-          - --exit-zero
-        verbose: true
-      - id: pylint_odoo
-        args:
-          - --rcfile=.pylintrc-mandatory

Diff for: .prettierrc.yml

@@ -1,8 +1,8 @@
-# Defaults for all prettier-supported languages.
 # Prettier will complete this with settings from .editorconfig file.
 bracketSpacing: false
-printWidth: 88
+printWidth: 119
 proseWrap: always
 semi: true
 trailingComma: "es5"
 xmlWhitespaceSensitivity: "strict"
+xmlSelfClosingSpace: true