Keep stress-test figures local; enforce via gitignore and git hooks - #31
Merged
Conversation
Figures and generated results under test/stress/ must not be committed to this public repo. Adds .gitignore rules for test/stress figures/results, a pre-commit hook that refuses to stage image files under test/stress/, and a pre-push guard that refuses to push if any are tracked. Also untracks the existing stress_test_results.png (the file stays local). Hooks run via core.hooksPath=scripts (set by npm postinstall).
Server-side complement to the local hooks: CI fails if any image file is tracked under test/stress, so figures cannot land on main even if a local hook is bypassed or not configured.
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
1 participant
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
Stress-test figures and generated data under
test/stress/must stay local and never reach this public repo. Enforced at three layers:.gitignore— ignorestest/stress/figures/,test/stress/results/, and image files undertest/stress/.scripts/pre-commit(new) — refuses to commit image files staged undertest/stress/, even if force-added.scripts/pre-push— refuses to push if any image files are tracked undertest/stress/.Also untracks the pre-existing
test/stress/stress_test_results.png(the file stays on disk locally). Hooks run via the repo'score.hooksPath=scripts(set bynpm postinstall).Note:
stress_test_results.pngremains in earlier git history (already public); this removes it from the current tree only.