Code review comment for ~mitchdz/ubuntu/+source/apache2:noble-add-ubuntu-logo

Revision history for this message
Mitchell Dzurick (mitchdz) wrote (last edit ):

I agree pipefail should be used here. same for `set -e`, I'm not the biggest fan of it, but kept it in for consistency across test scripts.

Since 2 tests do use bash (default-mods && run-test-suite) I'll change the interpreter to use bash and use pipefail to alleviate those concerns.

« Back to merge proposal