Skip to content

Commit

Permalink
yq fix for journal release script
Browse files Browse the repository at this point in the history
  • Loading branch information
lauterman committed Jan 24, 2025
1 parent d2a6942 commit 61e091d
Showing 1 changed file with 6 additions and 3 deletions.
9 changes: 6 additions & 3 deletions bin/pr-release-journals
Original file line number Diff line number Diff line change
Expand Up @@ -15,12 +15,15 @@ manifest="charts/journals/values.yaml"
# replace blank lines with marker
sed -i '/^$/s// #BLANK_LINE/' $manifest

# update image tag
yq --style=double w $manifest image.tag $CIRCLE_TAG -i
# sometimes this works
# yq --style=double w $manifest image.tag $CIRCLE_TAG -i

# some versions of yq use this
# sometimes this works
# yq -i ".image.tag=\"$CIRCLE_TAG\"" "$manifest"

# copilot suggested this
yq eval '.image.tag = strenv(CIRCLE_TAG)' -i $manifest

# restore blank lines
sed -i "s/ *#BLANK_LINE//g" $manifest

Expand Down

0 comments on commit 61e091d

Please sign in to comment.