Do not run prettier with pre-commit (#3196)

To fix issues with https://github.com/woodpecker-ci/woodpecker/pull/3190

Prettier still runs at woodpecker.

Also fix the duplication in our CI.

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

.pre-commit-config.yaml

@@ -54,5 +54,5 @@ ci:
   autoupdate_commit_msg: '[pre-commit.ci] pre-commit autoupdate'
   autoupdate_schedule: monthly
   # NB: hadolint not included in pre-commit.ci
-  skip: [check-hooks-apply, check-useless-excludes, hadolint]
+  skip: [check-hooks-apply, check-useless-excludes, hadolint, prettier]
   submodules: false

.prettierignore

@@ -1,8 +1,10 @@
-pnpm-lock.yaml
 build/
 docs/versioned_docs/
 docs/.docusaurus/
+docs/pnpm-lock.yaml
 dist/
-web/components.d.ts
 CHANGELOG.md
-web/src/assets/dayjsLocales/
+
+# web/ must be directly formatted from there
+# to prevent conflicts with different prettier version
+web/

.woodpecker/test.yaml

@@ -71,7 +71,9 @@ steps:
   lint-editorconfig:
     image: docker.io/mstruebing/editorconfig-checker:2.7.2
     when:
-      event: pull_request
+      - event: pull_request
+      - event: push
+        branch: renovate/*
 
   lint-license-header:
     image: *golang_image
@@ -82,7 +84,10 @@ steps:
 
   prettier:
     image: docker.io/woodpeckerci/plugin-prettier:next
-    when: *when
+    when:
+      - event: pull_request
+      - event: push
+        branch: renovate/*
 
   test:
     depends_on:

web/.prettierignore

@@ -4,5 +4,5 @@ coverage/
 LICENSE
 components.d.ts
 src/assets/locales/*.json
-src/assets/timeAgoLocales/
+src/assets/dayjsLocales/
 !src/assets/locales/en.json