Skip to content

Commit 5de5937

Browse files
fix: release workflow CodeQL query-filter and smoke --force-reinstall (#121)
- Switch CodeQL config from paths-ignore to query-filters exclude for py/weak-sensitive-data-hashing (paths-ignore doesn't block cross-file data-flow findings) - Add --force-reinstall to smoke pip install so system-site-packages venv doesn't skip reinstall when engraphis is pre-installed
1 parent 2b517f8 commit 5de5937

2 files changed

Lines changed: 4 additions & 3 deletions

File tree

.github/codeql/codeql-config.yml

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10,5 +10,6 @@
1010

1111
name: "Engraphis CodeQL config"
1212

13-
paths-ignore:
14-
- engraphis/backends/embedder_deterministic.py
13+
query-filters:
14+
- exclude:
15+
id: py/weak-sensitive-data-hashing

.github/workflows/release.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -100,7 +100,7 @@ jobs:
100100
index=$((index + 1))
101101
venv="$RUNNER_TEMP/engraphis-artifact-smoke-$index"
102102
python -m venv --system-site-packages "$venv"
103-
"$venv/bin/python" -m pip install --no-deps "$artifact"
103+
"$venv/bin/python" -m pip install --force-reinstall --no-deps "$artifact"
104104
(
105105
cd "$RUNNER_TEMP"
106106
"$venv/bin/python" - <<'PY'

0 commit comments

Comments
 (0)