Skip to content

Commit

Permalink
fix: Install semgrep; don't fail on existing tmp dir
Browse files Browse the repository at this point in the history
Fail on errors, too. This script was silently failing on missing semgrep.
  • Loading branch information
timmc-edx committed Aug 11, 2023
1 parent 921ba83 commit c170f69
Showing 1 changed file with 7 additions and 1 deletion.
8 changes: 7 additions & 1 deletion scripts/update_setup_py.sh
Original file line number Diff line number Diff line change
Expand Up @@ -7,9 +7,15 @@
# message for complying with conventional commit guidelines.
# Date: 11/01/2021

set -eu -o pipefail

script_home=$(dirname -- "${BASH_SOURCE[0]}")

mkdir update-setup-tmp
# Need semgrep, and also want to ensure we're only installing it into
# a virtualenv and not unexpectedly into the user's home dir.
"$VIRTUAL_ENV"/bin/pip install semgrep

mkdir -p update-setup-tmp

# generate and store requires.txt file for future comparison
python setup.py bdist_wheel
Expand Down

0 comments on commit c170f69

Please sign in to comment.