Skip to content

Fix git-lfs missing check in run_dev.sh#238

Open
Exokern wants to merge 1 commit into
NVIDIA-ISAAC-ROS:release-3.2from
Exokern:fix/release-3-2-run-dev-git-lfs-check
Open

Fix git-lfs missing check in run_dev.sh#238
Exokern wants to merge 1 commit into
NVIDIA-ISAAC-ROS:release-3.2from
Exokern:fix/release-3-2-run-dev-git-lfs-check

Conversation

@Exokern
Copy link
Copy Markdown

@Exokern Exokern commented May 5, 2026

Summary

  • keep the git-lfs prerequisite check inside an if ! ...; then guard so set -e does not exit before print_error runs
  • fix the insalled typo in the user-facing error message

Fixes #151
Fixes #167
Fixes #176

Validation

  • bash -n scripts/run_dev.sh
  • git diff --check

I did not run the full Docker launch path locally; this change only adjusts the preflight guard around the existing git lfs check.

Signed-off-by: MichaelLud1AI <79793838+MichaelLud1AI@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant