Skip to content

Commit

Permalink
fix(docker): fix incorrect version being passed in during docker build
Browse files Browse the repository at this point in the history
  • Loading branch information
azlam committed Dec 16, 2024
1 parent ca7337e commit 68b6447
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 3 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/sfp-build-docker.yml
Original file line number Diff line number Diff line change
Expand Up @@ -88,7 +88,7 @@ jobs:
tags: |
ghcr.io/${{ inputs.repo }}/${{ inputs.image }}:${{ env.PKG_VERSION }}-${{ inputs.suffix-tag }}
build-args: |
SFPOWERSCRIPTS_VERSION=${{ env.PKG_VERSION }}
SFP_VERSION=${{ env.PKG_VERSION }}
GIT_COMMIT=${{ github.sha}}
Expand Down
3 changes: 2 additions & 1 deletion dockerfiles/sfp-lite.Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@ FROM ubuntu:24.04

ARG GIT_COMMIT
ARG NODE_MAJOR=22
ARG SFP_VERSION

LABEL org.opencontainers.image.description "sfp is a build system for modular development in Salesforce."
LABEL org.opencontainers.image.licenses "MIT"
Expand Down Expand Up @@ -56,7 +57,7 @@ RUN mkdir -p /etc/apt/keyrings \

# Install sfp
RUN npm install --global --omit=dev \
@flxbl-io/sfp@${SFPOWERSCRIPTS_VERSION}
@flxbl-io/sfp@${SFP_VERSION}

WORKDIR /root

Expand Down
3 changes: 2 additions & 1 deletion dockerfiles/sfp.Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@ ARG BROWSERFORCE_VERSION=4.5.0
ARG SFDMU_VERSION=4.37.0
ARG GIT_COMMIT
ARG NODE_MAJOR=22
ARG SFP_VERSION

LABEL org.opencontainers.image.description "sfp is a build system for modular development in Salesforce."
LABEL org.opencontainers.image.licenses "MIT"
Expand Down Expand Up @@ -89,7 +90,7 @@ RUN npm install --global yarn --omit-dev \
# Install SF cli and sfpowerscripts
RUN npm install --global --omit=dev \
@salesforce/cli@${SF_CLI_VERSION} \
@flxbl-io/sfp@${SFPOWERSCRIPTS_VERSION} \
@flxbl-io/sfp@${SFP_VERSION} \
&& npm cache clean --force


Expand Down

0 comments on commit 68b6447

Please sign in to comment.