Repository navigation
Move run logs out of the repository - #240
Open
michaellchung wants to merge 2 commits into
Open
michaellchung wants to merge 2 commits into
michaellchung wants to merge 2 commits into
Conversation
Untrack 236 run-log files (run-logs/, *.log, *.jsonl, *.out, *.err) under openspec/changes/*. Copies live in infra-drafts/evidence/<change>/; each old directory keeps an EVIDENCE.md pointer. Drop log re-include lines from .gitignore and add trailing log rules. Add tests/test_no_tracked_logs.py; test_fn_learning_signal reads the try27 log from the out-of-repo evidence dir. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…bsent The run logs this test reads were moved out of the repository, so in a clean checkout the arm discovery raises FileNotFoundError instead of skipping. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Moves committed run logs out of the repository (PM rule: run logs must not live in git).
openspec/changes/*/(jsonl / log outputs from past GPU runs). The files are kept outside the repo; each of the 10 affected directories keeps anEVIDENCE.mdpointer.tests/test_no_tracked_logs.py, which fails if a run log is committed again. Test inputs undertests/anddata/are allowed.test_rl_engine_equivalence.py: skips the committed-evidence check when its run logs are absent, instead of raisingFileNotFoundErroron a clean checkout.This is the same change as the first commits of #239, split out so it can merge on its own, independent of #238. #239 will drop these commits after this PR merges.
Testing
Ran the 142 test files that reference run logs or
openspec/changes/paths (yeto-test-venv, excluding mismatch_tape / upstream_parse_args) on this branch and onmain:main: 16 failed, 2757 passed, 19 skipped, 16 errorsmain). The extra passes are the newtest_no_tracked_logs.py.🤖 Generated with Claude Code