Skip to content

Commit cf1b60d

Browse files
committed
Improve argument building in sync-uv action
Refactored the scripts for building group and extra arguments to use explicit if statements for non-empty values and added 'set -x' for debugging. Also added '|| exit 0' to the output commands to prevent failures if output writing fails.
1 parent 3e34cc1 commit cf1b60d

File tree

1 file changed

+24
-18
lines changed

1 file changed

+24
-18
lines changed

.github/actions/sync-uv/action.yml

Lines changed: 24 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -31,29 +31,35 @@ runs:
3131
shell: bash
3232
id: groups
3333
run: |
34-
args=""
35-
if [ -n "${{ inputs.groups }}" ]; then
36-
IFS=',' read -ra GROUPS <<< "${{ inputs.groups }}"
37-
for group in "${GROUPS[@]}"; do
38-
group=$(echo "$group" | xargs)
39-
[ -n "$group" ] && args="$args --group $(printf %q "$group")"
40-
done
41-
fi
42-
echo "value=$args" >> $GITHUB_OUTPUT
34+
set -x
35+
args=""
36+
if [ -n "${{ inputs.groups }}" ]; then
37+
IFS=',' read -ra GROUPS <<< "${{ inputs.groups }}"
38+
for group in "${GROUPS[@]}"; do
39+
group=$(echo "$group" | xargs)
40+
if [ -n "$group" ]; then
41+
args="$args --group $(printf %q "$group")"
42+
fi
43+
done
44+
fi
45+
echo "value=$args" >> $GITHUB_OUTPUT || exit 0
4346
4447
- name: "Build extras args"
4548
shell: bash
4649
id: extras
4750
run: |
48-
args=""
49-
if [ -n "${{ inputs.extras }}" ]; then
50-
IFS=',' read -ra EXTRAS <<< "${{ inputs.extras }}"
51-
for extra in "${EXTRAS[@]}"; do
52-
extra=$(echo "$extra" | xargs)
53-
[ -n "$extra" ] && args="$args --extra $(printf %q "$extra")"
54-
done
55-
fi
56-
echo "value=$args" >> $GITHUB_OUTPUT
51+
set -x
52+
args=""
53+
if [ -n "${{ inputs.extras }}" ]; then
54+
IFS=',' read -ra EXTRAS <<< "${{ inputs.extras }}"
55+
for extra in "${EXTRAS[@]}"; do
56+
extra=$(echo "$extra" | xargs)
57+
if [ -n "$extra" ]; then
58+
args="$args --extra $(printf %q "$extra")"
59+
fi
60+
done
61+
fi
62+
echo "value=$args" >> $GITHUB_OUTPUT || exit 0
5763
5864
- name: "Run UV sync"
5965
shell: bash

0 commit comments

Comments
 (0)