Skip to content

Commit

Permalink
Merge pull request #42 from kustomize-everything/fix-yq-change
Browse files Browse the repository at this point in the history
fix: remove unneeded regex sub (fixes #41)
  • Loading branch information
navneet-bairwal authored May 16, 2024
2 parents 46cb4f3 + 29f4545 commit 67457ce
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 4 deletions.
3 changes: 0 additions & 3 deletions action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -95,9 +95,6 @@ runs:
version: ${{ inputs.helm-version }} # default is latest (stable)
id: install

- name: Install yq
uses: mikefarah/yq@v4.42.1

- name: Set Git Author
shell: bash
run: |
Expand Down
2 changes: 1 addition & 1 deletion kustomize-build.sh
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@ pushd "${RENDER_DIR}" || exit 1
if [[ -s "${RENDER_FILE}" ]]; then
# Split the rendered file into individual files for each resource
# Invalid GitHub artifact path name characters: Double quote ", Colon :, Less than <, Greater than >, Vertical bar |, Asterisk *, Question mark ?
yq -s '.kind + "-" + (.apiVersion | sub("/", "_")) + "-" + (.metadata.name | sub("[:<>|*?/\\]", "_")) + ".yaml"' < "${RENDER_FILE}"
yq -s '.kind + "-" + (.apiVersion | sub("/", "_")) + "-" + .metadata.name + ".yaml"' < "${RENDER_FILE}"

if is_debug; then
echo "[debug] ls ${RENDER_DIR} post-yq"
Expand Down

0 comments on commit 67457ce

Please sign in to comment.