Skip to content

Commit c8c5496

Browse files
author
Ovtcharov
committed
fix(tests): pin skill-audit digest fixtures to LF so the vectors hold on Windows
content_digest hashes fixture bytes raw, with no line-ending normalization, because it covers arbitrary content and not just text. The fixture tree carried no .gitattributes pin, so a Windows checkout with core.autocrlf=true rewrote tree/SKILL.md, tree/tools.py and tree/scripts/run.py to CRLF and changed the hash — test_content_digest_of_the_fixture_tree_matches_the_vector and test_content_digest_ignores_a_build_cache failed on Windows only while CI stayed green on Linux. -text on the fixture directory stops conversion in both directions, so the LF files stay LF and skill_md_crlf.txt keeps the CRLF bytes it deliberately tests.
1 parent a696a06 commit c8c5496

1 file changed

Lines changed: 4 additions & 0 deletions

File tree

.gitattributes

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,10 @@
22
cpp/tests/fixtures/*.png binary
33
cpp/tests/fixtures/*.jpg binary
44

5+
# content_digest hashes these fixture bytes raw (#2468), so an autocrlf checkout
6+
# would change the hash and fail the vectors on Windows only.
7+
tests/fixtures/skill_audit_digest/** -text
8+
59
# Every email-agent PR appends an [Unreleased] bullet to this changelog;
610
# without a union merge, sibling PRs merged close together conflict on the
711
# same insertion point.

0 commit comments

Comments
 (0)