Skip to content

Commit

Permalink
Merge pull request #12 from IdanKoblik/11-cicd-fix
Browse files Browse the repository at this point in the history
CICD fix
  • Loading branch information
IdanKoblik authored Oct 3, 2024
2 parents 84ea34d + 76887fa commit e1f746c
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 4 deletions.
5 changes: 2 additions & 3 deletions .github/workflows/callable.tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -21,10 +21,9 @@ jobs:
uses: actions/setup-node@v2
with:
node-version: ${{ matrix.node-version }}
cache: 'npm'


- name: Install Node.js dependencies
run: npm ci
run: npm i

- name: Run tests
run: npm test
7 changes: 6 additions & 1 deletion Dockerfile
Original file line number Diff line number Diff line change
@@ -1,6 +1,11 @@
ARG NODE_VERSION=18.0.0
FROM node:${NODE_VERSION}-alpine

RUN apk add --no-cache --update curl ca-certificates openssl git tar bash sqlite fontconfig \
&& adduser --disabled-password --home /home/container container

LABEL org.opencontainers.image.authors="idankob@gmail.com"

ENV NODE_ENV production

WORKDIR /home/container
Expand All @@ -9,7 +14,7 @@ COPY package*.json ./
RUN npm install
RUN npm ci --omit=dev

USER node
USER container

COPY . .

Expand Down

0 comments on commit e1f746c

Please sign in to comment.