Skip to content

Commit

Permalink
Ensure enabling local symbols doesn't change the module hash (aws#2169)
Browse files Browse the repository at this point in the history
### Description of changes:
The original intention of aws#1005 was to
ensure break-kat.go always worked with the library.
aws#1252 then used a side effect to get
the module hash but didn't actually fail the build if the hash was
different.

Turn on `pipefail` so the script exits unsuccessfully if any command
fails even in a pipe. Previously if test-break-kat.sh failed the script
didn't exit, instead it would continue to the grep which would also fail
to find any matching string, and then the `|| true` ensured the script
always continued on.

By submitting this pull request, I confirm that my contribution is made
under the terms of the Apache 2.0 license and the ISC license.
  • Loading branch information
andrewhop committed Feb 6, 2025
1 parent a95190e commit 593559e
Show file tree
Hide file tree
Showing 5 changed files with 206 additions and 138 deletions.
Loading

0 comments on commit 593559e

Please sign in to comment.