Skip to content

Commit

Permalink
fix(build-system): undefined IMAGE_TAG and ARG_TAG (#2030)
Browse files Browse the repository at this point in the history
  • Loading branch information
ludamad authored Sep 5, 2023
1 parent a0ccd4a commit dfdba4b
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 4 deletions.
4 changes: 1 addition & 3 deletions build-system/scripts/deploy_dockerhub
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@ if [ -z "$COMMIT_TAG" ]; then
fi

REPOSITORY=$1
IMAGE_TAG=$COMMIT_TAG_VERSION
IMAGE_COMMIT_URI=$ECR_DEPLOY_URL/$REPOSITORY:$COMMIT_HASH
IMAGE_DEPLOY_URI=aztecprotocol/$REPOSITORY:$IMAGE_TAG
IMAGE_LATEST_URI=aztecprotocol/$REPOSITORY:latest
Expand Down Expand Up @@ -34,9 +35,6 @@ if [ -z "$VERSION" ]; then
exit 1
fi

IMAGE_TAG=$COMMIT_TAG_VERSION


echo "Deploying to dockerhub: $IMAGE_DEPLOY_URI"

# Login.
Expand Down
2 changes: 1 addition & 1 deletion build-system/scripts/deploy_npm
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@ if [ -n "$TAG" ]; then
TAG_ARG="--tag $TAG"
fi

readonly PUBLISHED_VERSION=$(npm show . version $TAG_ARG 2> /dev/null)
readonly PUBLISHED_VERSION=$(npm show . version ${TAG_ARG:-} 2> /dev/null)
readonly HIGHER_VERSION=$(npx semver ${VERSION} ${PUBLISHED_VERSION} | tail -1)

# If there is already a published package equal to given version, assume this is a re-run of a deploy, and early out.
Expand Down

0 comments on commit dfdba4b

Please sign in to comment.