Skip to content

Commit

Permalink
shellcheck: SC2028 (info): echo may not expand escape sequences. Use …
Browse files Browse the repository at this point in the history
…printf.
  • Loading branch information
esolitos authored and gitbutler-client committed Oct 2, 2024
1 parent cdb1340 commit 6b5f9e6
Showing 1 changed file with 3 additions and 7 deletions.
10 changes: 3 additions & 7 deletions .github/scripts/delete-old-cloudsmith-artifacts.sh
Original file line number Diff line number Diff line change
Expand Up @@ -54,10 +54,7 @@ done

echo
if [[ "${GITHUB_EVENT_NAME}" == "schedule" || "${GITHUB_EVENT_NAME}" == "workflow_dispatch" ]]; then
echo '\033[1;33mDeleting the following RPMs:\033[0m'
echo
cat "${RPMS_TO_DELETE}"
echo
printf '\033[1;33mDeleting the following RPMs:\033[0m\n\n' "${RPMS_TO_DELETE}"

while IFS= read -r RPMID
do
Expand All @@ -68,8 +65,7 @@ if [[ "${GITHUB_EVENT_NAME}" == "schedule" || "${GITHUB_EVENT_NAME}" == "workflo
"https://api.cloudsmith.io/v1/packages/vespa/open-source-rpms/${RPMID}/"
done < "${RPMS_TO_DELETE}"
else
echo '\033[1;32m#### Dry-run mode ####\033[0m'
echo '\033[0;32mWould have deleted the following RPMs:\033[0m'
echo
printf '\033[1;32m#### Dry-run mode ####\033[0m\n'
printf '\033[0;32mWould have deleted the following RPMs:\033[0m\n\n'
cat "${RPMS_TO_DELETE}"
fi

0 comments on commit 6b5f9e6

Please sign in to comment.