Skip to content

Commit

Permalink
[FIX] prettier for v13 broken in mirror
Browse files Browse the repository at this point in the history
https://github.com/pre-commit/mirrors-prettier/issues/6 affects our repos, as it can be seen on https://github.com/OCA/social/runs/1770760744?check_suite_focus=true#step:4:110 for OCA/social#664 and it produces random results when running `pre-commit` in different environments.

I apply here the workaround explained in that issue.

At the same time, I double-checked that we can run all prettier jobs in a single hook, so I take this chance to apply that logic, and to limit by file extensions, to make it work just like the v14 branch with #48, more predictable and efficent.
  • Loading branch information
Jairo Llopis authored and yajo committed Jan 27, 2021
1 parent 2a361ba commit aaff28d
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions version-specific/13.0/.pre-commit-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -47,12 +47,12 @@ repos:
rev: v1.19.1
hooks:
- id: prettier
- id: prettier
name: prettier xml plugin
name: prettier (with plugin-xml)
entry: prettier --write --list-different
additional_dependencies:
- "[email protected]"
- "@prettier/[email protected]"
files: \.xml$
files: \.(css|htm|html|js|json|jsx|less|md|scss|toml|ts|xml|yaml|yml)$
- repo: https://github.com/pre-commit/mirrors-eslint
rev: v6.8.0
hooks:
Expand Down

0 comments on commit aaff28d

Please sign in to comment.