Skip to content

Commit f49df15

Browse files
committed
fix(turtlebot3): Fix shellcheck issues
- Remove unused YELLOW variable from check-entities.sh - Add -r flag to read command in restore-normal.sh to handle backslashes properly
1 parent 196d939 commit f49df15

2 files changed

Lines changed: 1 addition & 2 deletions

File tree

demos/turtlebot3_integration/check-entities.sh

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,6 @@ API_BASE="${GATEWAY_URL}/api/v1"
88
# Colors for output
99
BLUE='\033[0;34m'
1010
GREEN='\033[0;32m'
11-
YELLOW='\033[1;33m'
1211
NC='\033[0m'
1312

1413
echo_step() {

demos/turtlebot3_integration/restore-normal.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ echo "Canceling any active navigation goals..."
2626
# List active executions and cancel them
2727
EXECUTIONS=$(curl -s "${API_BASE}/apps/bt-navigator/operations/navigate_to_pose/executions" 2>/dev/null)
2828
if echo "$EXECUTIONS" | jq -e '.items[]' > /dev/null 2>&1; then
29-
echo "$EXECUTIONS" | jq -r '.items[].id' | while read EXEC_ID; do
29+
echo "$EXECUTIONS" | jq -r '.items[].id' | while read -r EXEC_ID; do
3030
if [ -n "$EXEC_ID" ]; then
3131
curl -s -X DELETE "${API_BASE}/apps/bt-navigator/operations/navigate_to_pose/executions/$EXEC_ID" > /dev/null 2>&1
3232
echo " Canceled execution: $EXEC_ID"

0 commit comments

Comments
 (0)