Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

run_dev.sh: Enable tests to fail gracefully #159

Open
wants to merge 1 commit into
base: release-3.1
Choose a base branch
from

Conversation

bsb808
Copy link

@bsb808 bsb808 commented Nov 22, 2024

Issue

When following the issac_ros_nvblox quickstart tutorial the verification of host setup (e.g., checking to see if git-lfs is installed) fail silently. This is because of the set -e fail fast option which exits before the output to stdout.

Fix

I added edits to two of the obvious checks that will fail silently enabling them to provide the desired reporting to stdout.

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