From 5409b98da1018f08e8a1196e2203dd86079982f7 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Julian=20T=C3=B6lle?= <julian.toelle97@gmail.com>
Date: Sat, 11 Jun 2022 18:50:12 +0200
Subject: [PATCH] fix(ci): ignore failing peer resolution

Some 3rd party nest packages depend on Nest 7 while we use 8. In practice
this still works, but the usual dependency resolution fails and requires
an explicit override.

This should also unblock auto-merging for renovate, as that currently fails
in CI on the `npm ci` step.
---
 .github/workflows/ci.yaml | 4 ++--
 Dockerfile                | 4 ++--
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/.github/workflows/ci.yaml b/.github/workflows/ci.yaml
index 128a42f..0cafa14 100644
--- a/.github/workflows/ci.yaml
+++ b/.github/workflows/ci.yaml
@@ -22,7 +22,7 @@ jobs:
         with:
           node-version: 16
 
-      - run: npm ci
+      - run: npm ci --legacy-peer-deps
       - run: npm run build
       - run: npm run lint:api
       - run: npm run test:cov
@@ -51,7 +51,7 @@ jobs:
         with:
           node-version: 16
 
-      - run: npm ci
+      - run: npm ci --legacy-peer-deps
 
       - run: npm ci
         working-directory: frontend
diff --git a/Dockerfile b/Dockerfile
index 84b9593..812ec36 100644
--- a/Dockerfile
+++ b/Dockerfile
@@ -22,7 +22,7 @@ COPY frontend/*.json /app/frontend/
 FROM common as build-api
 LABEL stage="build-api"
 
-RUN npm ci
+RUN npm ci --legacy-peer-deps
 
 COPY src/ /app/src/
 RUN NODE_ENV=production npm run build
@@ -52,7 +52,7 @@ RUN NODE_ENV=production npm run build
 FROM common as app
 LABEL stage="app"
 
-RUN npm ci --only=production
+RUN npm ci --omit=dev --legacy-peer-deps
 COPY --from=build-api /app/dist/ /app/dist/
 COPY --from=build-frontend /app/frontend/build /app/static/