open-telemetry / opentelemetry-cpp

The OpenTelemetry C++ Client
https://opentelemetry.io/
Apache License 2.0
817 stars 392 forks source link

[MISC] Use set -e on all shell scripts and pass shellcheck --severity=error #2616

Closed johnbley closed 3 months ago

johnbley commented 3 months ago

Changes

Numerous shell scripts were not using set -e or equivalent. Additionally, make all scripts pass at least shellcheck --severity=error (basically just argument array quoting and exit values). I can add an additional PR for using shellcheck in CI.