Skip to content

Commit 793e02e

Browse files
committed
update result to node 10 and latest npm packages
The result app was years out of date with node and npm packages, and required multiple changes to use the current supported socket.io and postgres db libs. Minimal changes to server.js to support change in pg syntax. A few minor changes to Dockerfile to improve readability, conform to docker best practices, and change to use npm ci to ensure future builds don't break again due to difference between package and package-lock.
1 parent 368968c commit 793e02e

File tree

5 files changed

+883
-7005
lines changed

5 files changed

+883
-7005
lines changed

result/Dockerfile

Lines changed: 8 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,18 +1,19 @@
1-
FROM node:8.9-alpine
1+
FROM node:10-alpine
22

3-
RUN mkdir -p /app
43
WORKDIR /app
54

65
RUN npm install -g nodemon
7-
RUN npm config set registry https://registry.npmjs.org
8-
COPY package.json /app/package.json
9-
RUN npm install \
10-
&& npm ls \
6+
7+
COPY package*.json ./
8+
9+
RUN npm ci \
1110
&& npm cache clean --force \
1211
&& mv /app/node_modules /node_modules
13-
COPY . /app
12+
13+
COPY . .
1414

1515
ENV PORT 80
16+
1617
EXPOSE 80
1718

1819
CMD ["node", "server.js"]

0 commit comments

Comments
 (0)