Fix Dockerfile - actually install TS

lowercasename · Jul 31, 2024 · 6ceae29 · 6ceae29
1 parent 43d0af0
commit 6ceae29
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 6 deletions.
diff --git a/.github/workflows/publish-ghcr.yaml b/.github/workflows/publish-ghcr.yaml
@@ -15,11 +15,6 @@ jobs:
       - name: Checkout
         uses: actions/checkout@v4
 
-      - name: Install pnpm
-        uses: pnpm/action-setup@v4
-        with:
-          version: 9
-
       - name: Publish to GHCR
         run: echo ${{ secrets.GITHUB_TOKEN }} | docker login ghcr.io -u ${{ github.actor }} --password-stdin
       - run: docker build -t ghcr.io/${{ github.repository }}:latest .

diff --git a/Dockerfile b/Dockerfile
@@ -1,9 +1,10 @@
 FROM node:20-alpine
+ENV NODE_ENV=production
 WORKDIR /app
 RUN apk add --no-cache python3 build-base
 ADD package.json pnpm-lock.yaml /app/
 RUN npm install -g pnpm
-RUN pnpm install --prod
+RUN pnpm install
 COPY . /app/
 # Always exit 0 here because TSC will fail while we're migrating to TypeScript but
 # not everything uses TypeScript