Speedup danger-review by caching node_modules instead of .yarn-cache

Signed-off-by: Rémy Coutable <remy@rymai.me>

.gitlab/ci/global.gitlab-ci.yml

@@ -29,6 +29,15 @@
       - vendor/gitaly-ruby
     policy: pull
 
+.yarn-cache:
+  cache:
+    key:
+      files:
+        - yarn.lock
+      prefix: "v1"
+    paths:
+      - node_modules/
+
 .use-pg9:
   image: "registry.gitlab.com/gitlab-org/gitlab-build-images:ruby-2.6.6-golang-1.14-git-2.26-lfs-2.9-chrome-73.0-node-12.x-yarn-1.21-postgresql-9.6-graphicsmagick-1.3.34"
   services:

.gitlab/ci/review.gitlab-ci.yml

@@ -227,13 +227,12 @@ parallel-spec-reports:
 danger-review:
   extends:
     - .default-retry
-    - .default-cache
+    - .yarn-cache
     - .review:rules:danger
   image: registry.gitlab.com/gitlab-org/gitlab-build-images:danger
   stage: test
   needs: []
   script:
-    - git version
-    - node --version
-    - yarn install --frozen-lockfile --cache-folder .yarn-cache --prefer-offline
+    - source scripts/utils.sh
+    - retry yarn install --frozen-lockfile
     - danger --fail-on-errors=true --verbose