Skip to content

Commit

Permalink
Don't do an unnecessary build
Browse files Browse the repository at this point in the history
  • Loading branch information
xpdota committed Jun 16, 2024
1 parent e658cb1 commit 4e78ad3
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 15 deletions.
11 changes: 0 additions & 11 deletions .github/workflows/backend_docker.yml
Original file line number Diff line number Diff line change
Expand Up @@ -18,17 +18,6 @@ jobs:
steps:
- uses: actions/checkout@v3

- name: Use Node.js 20.x
uses: actions/setup-node@v3
with:
node-version: 20.x

- name: Build
run: |
npm ci
npm run build
npm run test
# Docker stuff begins here
- name: Set up QEMU
uses: docker/setup-qemu-action@v3
Expand Down
8 changes: 4 additions & 4 deletions packages/backend-resolver/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -2,15 +2,15 @@ FROM node:20 as build

WORKDIR /build
COPY . .
RUN npm install
RUN npm ci
# TODO: this step specifically isn't really doing anything, since it's run from the TS files
RUN npm run build
RUN npm --workspaces --if-present test

RUN npm run test

# To be run from project root
FROM node:20
WORKDIR /app
#COPY --from=build /build/packages/backend-resolver/build/src .
#COPY --from=build /build/node_modules ./node_modules
COPY --from=build /build /app
EXPOSE 30000
# -T == transpile only. Reduces memory usage.
Expand Down

0 comments on commit 4e78ad3

Please sign in to comment.