Remove redundant npm install from test-runner Dockerfile
Some checks failed
CI/CD / detect-changes (pull_request) Successful in 3s
CI/CD / test-frontend (pull_request) Successful in 6s
CI/CD / test-go-pg (pull_request) Successful in 2m43s
CI/CD / test-sqlite (pull_request) Successful in 3m2s
CI/CD / test-runners (pull_request) Failing after 1m12s
CI/CD / build-and-deploy (pull_request) Has been skipped
Some checks failed
CI/CD / detect-changes (pull_request) Successful in 3s
CI/CD / test-frontend (pull_request) Successful in 6s
CI/CD / test-go-pg (pull_request) Successful in 2m43s
CI/CD / test-sqlite (pull_request) Successful in 3m2s
CI/CD / test-runners (pull_request) Failing after 1m12s
CI/CD / build-and-deploy (pull_request) Has been skipped
The Playwright Docker image already ships with playwright and browsers pre-installed. The npm install was failing (exit code 1, hidden by 2>/dev/null) and blocking the build. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
@@ -29,7 +29,6 @@ services:
|
||||
context: .
|
||||
dockerfile_inline: |
|
||||
FROM mcr.microsoft.com/playwright:v1.52.0-noble
|
||||
RUN npm install --no-save playwright 2>/dev/null
|
||||
WORKDIR /work
|
||||
COPY ci/ /work/ci/
|
||||
RUN chmod +x /work/ci/*.sh
|
||||
|
||||
Reference in New Issue
Block a user