Skip to content

Commit

Permalink
- remove obsolete node-sass
Browse files Browse the repository at this point in the history
- add sass package that do not require binary deps (node-gyp_
- remove unused deps from dockerfile
- use LTS node version in Dockerfile
- use `npm ci` instead of npm i
  • Loading branch information
nosovk committed Nov 29, 2021
1 parent 0bf4c7a commit 1f73140
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 16 deletions.
2 changes: 2 additions & 0 deletions .dockerignore
Original file line number Diff line number Diff line change
@@ -1 +1,3 @@
node_modules
dist
.github
19 changes: 4 additions & 15 deletions Dockerfile
Original file line number Diff line number Diff line change
@@ -1,22 +1,11 @@
FROM node:15-alpine

RUN alias python=python3
RUN apk add python3
RUN apk add python2
RUN apk add make
RUN apk add gcc
RUN apk add g++
RUN apk add git
FROM node:16-alpine

WORKDIR /app

COPY package.json ./
RUN npm install
COPY package.json package-lock.json ./
RUN npm ci

COPY public/ public/
COPY src/ src/
COPY styles/ styles/
COPY webpack.config.js .babelrc ./
COPY . .

RUN npm run build

Expand Down
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@
"extract-text-webpack-plugin": "4.0.0-beta.0",
"file-loader": "6.0.0",
"html-webpack-plugin": "4.3.0",
"node-sass": "4.14.1",
"sass": "1.43.4",
"sass-loader": "8.0.2",
"style-loader": "1.2.1",
"webpack": "4.43.0",
Expand Down

0 comments on commit 1f73140

Please sign in to comment.