Merge pull request #212 from acsone/fix-gen-addon-readme-skip-sbi

Fix skipping of oca-gen-addon-readme in CI

Author: sbidoul

.github/workflows/lint.yml

@@ -14,12 +14,3 @@ jobs:
       - uses: actions/checkout@v3
       - uses: actions/setup-python@v2
       - uses: pre-commit/action@v2.0.0
-        env:
-          # Consider valid a PR that changes README fragments but doesn't
-          # change the README.rst file itself. It's not really a problem
-          # because the bot will update it anyway after merge. This way, we
-          # lower the barrier for functional contributors that want to fix the
-          # readme fragments, while still letting developers get README
-          # auto-generated (which also helps functionals when using runboat).
-          # DOCS https://pre-commit.com/#temporarily-disabling-hooks
-          SKIP: oca-gen-addon-readme

src/.github/workflows/pre-commit.yml.jinja

@@ -39,6 +39,15 @@ jobs:
         run: pip install pre-commit
       - name: Run pre-commit
         run: pre-commit run --all-files --show-diff-on-failure --color=always
+        env:
+          # Consider valid a PR that changes README fragments but doesn't
+          # change the README.rst file itself. It's not really a problem
+          # because the bot will update it anyway after merge. This way, we
+          # lower the barrier for functional contributors that want to fix the
+          # readme fragments, while still letting developers get README
+          # auto-generated (which also helps functionals when using runboat).
+          # DOCS https://pre-commit.com/#temporarily-disabling-hooks
+          SKIP: oca-gen-addon-readme
       - name: Check that all files generated by pre-commit are in git
         run: |
           newfiles="$(git ls-files --others --exclude-from=.gitignore)"