Merge pull request #170 from hellos3b/f/build

build: js files built before docker

bot/Dockerfile

@@ -4,15 +4,17 @@ FROM node:16 as builder
 WORKDIR /app
 
 RUN npm install --global pnpm@6
-COPY pnpm-lock.yaml .
-RUN pnpm fetch
+COPY package.json pnpm-lock.yaml ./
+RUN pnpm install --production
 
 # Copy files required to build
-ADD package*.json bsconfig.json tsconfig.json ./ 
-ADD src/ ./src/
 
-RUN pnpm install -r --offline 
-RUN pnpm build:re
-RUN pnpm build:ts
+# ADD package*.json bsconfig.json tsconfig.json ./ 
+# ADD src/ ./src/
+
+COPY build/ ./build/
+
+# RUN pnpm build:re
+# RUN pnpm build:ts
 
 ENTRYPOINT pnpm start:prod

bot/README.md

@@ -14,7 +14,14 @@ pnpm start
 
 ## Deploy Bot
 
-First, create a [Docker Context](https://docs.docker.com/engine/context/working-with-contexts/) using your ssh host as a target. Then, pretending I named my context "s3bby":
+The docker image copies over the built js files, so first build the entire project:
+
+```sh
+pnpm build:re
+pnpm build:ts
+```
+
+Next, create a [Docker Context](https://docs.docker.com/engine/context/working-with-contexts/) using your ssh host as a target. Then, pretending I named my context "s3bby":
 
 ```sh
 $env:DOCKER_CONTEXT="s3bby"