Skip to content

Commit 5295aab

Browse files
committed
fix: add shellcheck source directives for container-internal ROS scripts
Add '# shellcheck source=/dev/null' before sourcing ROS setup files in scripts that run inside the Docker container, suppressing SC1091 (not following source) info-level warnings.
1 parent bde7882 commit 5295aab

3 files changed

Lines changed: 6 additions & 0 deletions

File tree

demos/moveit_pick_place/scripts/inject-collision.sh

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,9 @@
55

66
set -e
77

8+
# shellcheck source=/dev/null
89
source /opt/ros/jazzy/setup.bash
10+
# shellcheck source=/dev/null
911
source /root/demo_ws/install/setup.bash
1012

1113
echo "🚫 Injecting COLLISION fault..."

demos/moveit_pick_place/scripts/inject-planning-failure.sh

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,9 @@
55

66
set -e
77

8+
# shellcheck source=/dev/null
89
source /opt/ros/jazzy/setup.bash
10+
# shellcheck source=/dev/null
911
source /root/demo_ws/install/setup.bash
1012

1113
echo "🚫 Injecting PLANNING FAILURE fault..."

demos/moveit_pick_place/scripts/restore-normal.sh

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,9 @@
44

55
set -e
66

7+
# shellcheck source=/dev/null
78
source /opt/ros/jazzy/setup.bash
9+
# shellcheck source=/dev/null
810
source /root/demo_ws/install/setup.bash
911

1012
GATEWAY_URL="${GATEWAY_URL:-http://localhost:8080}"

0 commit comments

Comments
 (0)