Skip to content

Commit bf3c77d

Browse files
committed
Dockerfile: Use npm install instead of ci because we don't use a lock file
Due to how the multi-arch build works (specifying arch-dependent things in the lock file), we remove the lock file before building a plugin's container. This means we cannot use ci (which needs a lock file). Signed-off-by: Joaquim Rocha <joaquim.rocha@microsoft.com>
1 parent 8a3a0a5 commit bf3c77d

File tree

1 file changed

+1
-6
lines changed

1 file changed

+1
-6
lines changed

Dockerfile

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -30,12 +30,7 @@ RUN echo "Cleaning arch-dependent files for plugin $PLUGIN..."; \
3030
# Install dependencies for the specified plugin
3131
RUN echo "Installing deps for plugin $PLUGIN..."; \
3232
cd /headlamp-plugins/$PLUGIN; \
33-
echo "Installing $ENVIRONMENT dependencies..."; \
34-
if [ "$ENVIRONMENT" = "production" ]; then \
35-
npm ci --omit=dev; \
36-
else \
37-
npm ci; \
38-
fi
33+
npm install
3934

4035
# Build the specified plugin
4136
RUN echo "Building plugin $PLUGIN..."; \

0 commit comments

Comments
 (0)