Skip to content

Commit 85e6d51

Browse files
committed
fix(verify): detect changed records with a direct diff (shallow-safe)
CI fetches main with --depth=1, so the three-dot origin/main...HEAD diff has no merge-base and silently returned nothing — every '--changed'/'pr' run saw 0 changed records and skipped Tiers 1-3. Use a direct two-tree diff (origin/main HEAD) which only needs both tips. Refs #1
1 parent 9cbda7b commit 85e6d51

1 file changed

Lines changed: 8 additions & 2 deletions

File tree

app/verify/cli.py

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -35,10 +35,16 @@ def _now_iso() -> str:
3535

3636

3737
def _changed_data_slugs() -> set[str]:
38-
"""Repo-relative data/ paths changed vs origin/main (for CI --changed)."""
38+
"""Repo-relative data/ paths changed vs origin/main (for CI --changed).
39+
40+
Direct two-tree diff (``origin/main HEAD``), NOT three-dot ``origin/main...HEAD``:
41+
CI fetches main shallow (``--depth=1``), so there is no merge-base and the
42+
three-dot form silently returns nothing. A direct tree diff only needs both
43+
commit tips, which are always present.
44+
"""
3945
try:
4046
out = subprocess.run(
41-
["git", "diff", "--name-only", "origin/main...HEAD", "--", "data/"],
47+
["git", "diff", "--name-only", "origin/main", "HEAD", "--", "data/"],
4248
capture_output=True, text=True, check=True,
4349
).stdout
4450
except Exception:

0 commit comments

Comments
 (0)