forked from boos/BoosAPI
fix: restore docker builder install deps (#261)
* fix: exclude runtime db config from backup and migration * fix: restore docker builder install deps
This commit is contained in:
+1
-1
@@ -10,7 +10,7 @@ RUN apt-get update \
|
|||||||
ENV PYTHON=/usr/bin/python3
|
ENV PYTHON=/usr/bin/python3
|
||||||
|
|
||||||
COPY package.json package-lock.json ./
|
COPY package.json package-lock.json ./
|
||||||
RUN --mount=type=cache,target=/root/.npm npm ci --omit=peer --no-audit --no-fund
|
RUN --mount=type=cache,target=/root/.npm npm ci --no-audit --no-fund
|
||||||
|
|
||||||
COPY . .
|
COPY . .
|
||||||
RUN npm run build:web && npm run build:server
|
RUN npm run build:web && npm run build:server
|
||||||
|
|||||||
@@ -26,7 +26,10 @@ describe('docker workflows', () => {
|
|||||||
it('keeps server docker builds isolated from desktop packaging dependencies', () => {
|
it('keeps server docker builds isolated from desktop packaging dependencies', () => {
|
||||||
const dockerfile = readFileSync(resolve(process.cwd(), 'docker/Dockerfile'), 'utf8');
|
const dockerfile = readFileSync(resolve(process.cwd(), 'docker/Dockerfile'), 'utf8');
|
||||||
|
|
||||||
expect(dockerfile).toContain('npm ci --omit=peer --no-audit --no-fund');
|
expect(dockerfile).toContain('npm ci --no-audit --no-fund');
|
||||||
|
expect(dockerfile).not.toContain('npm ci --omit=peer --no-audit --no-fund');
|
||||||
|
expect(dockerfile).not.toContain('npm ci --include=dev --omit=peer --no-audit --no-fund');
|
||||||
expect(dockerfile).toContain('RUN npm run build:web && npm run build:server');
|
expect(dockerfile).toContain('RUN npm run build:web && npm run build:server');
|
||||||
|
expect(dockerfile).toContain('npm prune --omit=dev --no-audit --no-fund');
|
||||||
});
|
});
|
||||||
});
|
});
|
||||||
|
|||||||
Reference in New Issue
Block a user