Add set -e to buildspecs with multiline commands

[davidh44]

Motivation and Context

In multi line commands, a subsequent successful command will mask a previous command that fails

Modifications

Add set -e to beginning of multi line commands

Testing

Tested in CodeBuild runs

[sonarqubecloud]

Quality Gate failed

Failed conditions
0.0% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

[S-Saranya1]

Do we need to add this to buildspecs/update-master-from-release.yml as well? It looks like it also has a multi-command script block. And for s3-regression-tests.yml, it is covered in another PR, correct?

[davidh44]

Do we need to add this to buildspecs/update-master-from-release.yml as well? It looks like it also has a multi-command script block. And for s3-regression-tests.yml, it is covered in another PR, correct?

update-master-from-release.yml is already handled with the OR operator in the mvn versions command
s3-regression-tests will be updated once the tests are fixed

[github-actions]

This pull request has been closed and the conversation has been locked. Comments on closed PRs are hard for our team to see. If you need more assistance, please open a new issue that references this one.