forked from open-quantum-safe/oqs-provider
-
Notifications
You must be signed in to change notification settings - Fork 1
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Fix a minor bug in the
runtests.sh
. (open-quantum-safe#216)
This commit fixes a minor bug in `scripts/runtests.sh`: this script is run with POSIX sh on linux machines, meaning that `[[` are not recognized. However, `[[ … ]]` was used. Because `set -e` was not set, the minor bug described above did not lead to a CI failure. This commit fixes the `[[ … ]]` by using a simple `if grep`. It also sets `set -e` at the top of the file in order to catch any potential future mistakes. Finally, this commit also fixes minor warnings raised by [ShellCheck]. To display those warnings, one can simply run `shellcheck scripts/runtests.sh`. [ShellCheck]: https://www.shellcheck.net/ Signed-off-by: Felipe Ventura <[email protected]>
- Loading branch information
Showing
1 changed file
with
63 additions
and
66 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters