Skip to content

Commit

Permalink
#546: Fix testing process that relied on implicitly finding venv
Browse files Browse the repository at this point in the history
  • Loading branch information
PhilMiller authored and mattw-nws committed Aug 8, 2023
1 parent 3d53cf0 commit cd422ad
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions .github/actions/ngen-build/action.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -183,8 +183,11 @@ runs:
- name: Build Targets
#cmake >= 3.15 can build multiple targets
run: |
# Build Targets
# Disable leak detection during test enumeration
export ASAN_OPTIONS=detect_leaks=false
# Activate venv so that test discovery run during build works
. .venv/bin/activate
cmake --build ${{ inputs.build-dir }} --target ${{ inputs.targets }} -- -j ${{ inputs.build-cores }}
shell: bash

0 comments on commit cd422ad

Please sign in to comment.