Skip to content

Commit 45b4812

Browse files
committed
Fix outputs
Signed-off-by: ZePan110 <ze.pan@intel.com>
1 parent e3f8087 commit 45b4812

1 file changed

Lines changed: 4 additions & 2 deletions

File tree

.github/workflows/pr-image-size.yml

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -37,17 +37,19 @@ jobs:
3737
merged_commit=$(git log -1 --format='%H')
3838
files=$(git diff --name-status --diff-filter=ARM ${{ github.event.pull_request.base.sha }} ${merged_commit} | awk '{print $2}' | grep -E 'Dockerfile$' || true)
3939
echo "files<<EOF" >> $GITHUB_OUTPUT
40-
echo "$files" >> $GITHUB_OUTPUT
40+
echo "$files" | tr '\0' '\n' >> $GITHUB_OUTPUT
4141
echo "EOF" >> $GITHUB_OUTPUT
4242
4343
- name: Build and check image sizes
4444
if: steps.changed-dockerfiles.outputs.files != ''
4545
run: |
4646
cd ${{github.workspace}}
47+
echo "files: ${{ steps.changed-dockerfiles.outputs.files }}"
4748
merged_commit=$(git log -1 --format='%H')
4849
set -e
4950
while read -r dockerfile; do
5051
[ -z "$dockerfile" ] && continue
52+
echo "$dockerfile"
5153
dir=$(dirname "$dockerfile")
5254
file=$(basename "$dockerfile")
5355
image_base="pr-image-size-base:$(echo $dir | tr '/' '-')"
@@ -75,4 +77,4 @@ jobs:
7577
echo "::warning::Image size change: $size_base -> $size_pr MB}' (diff: $diff MB)"
7678
echo "File $dockerfile resulted in a change in the image size from $size_base -> $size_pr MB" >> GITHUB_STEP_SUMMARY
7779
docker rmi "$image_base" "$image_pr"
78-
done <<< "${{ steps.changed-dockerfiles.outputs.files }}"
80+
done < <(echo "${{ steps.changed-dockerfiles.outputs.files }}" | tr '\n' '\0')

0 commit comments

Comments
 (0)