Skip to content

Remove tracked PDF and gitignore papers directory#138

Merged
igerber merged 1 commit intomainfrom
fix/remove-papers-from-git
Feb 8, 2026
Merged

Remove tracked PDF and gitignore papers directory#138
igerber merged 1 commit intomainfrom
fix/remove-papers-from-git

Conversation

@igerber
Copy link
Owner

@igerber igerber commented Feb 8, 2026

Summary

  • Remove papers/2508.21536v2.pdf from git tracking (was accidentally committed)
  • Add papers/ to .gitignore to prevent future PDF commits
  • Local paper files are preserved, just no longer version-controlled

Methodology references (required if estimator / math changes)

  • N/A — no methodology changes

Validation

  • Tests added/updated: No test changes (repo hygiene only)
  • Backtest / simulation / notebook evidence: N/A

Security / privacy

  • Confirm no secrets/PII in this PR: Yes

Generated with Claude Code

Academic paper PDFs should not be in version control. The PDF was
previously committed; this removes it from tracking while keeping
the local copy.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@github-actions
Copy link

github-actions bot commented Feb 8, 2026

Overall assessment: ✅ Looks good

Executive summary

  • No methodology, estimator, or statistical output changes in this PR.
  • Change is limited to repo hygiene: ignore papers/ and remove a tracked PDF.
  • No edge-case, inference, or control-group logic touched.
  • No tests required for this change.

Methodology

  • No findings. This PR only updates .gitignore and removes a binary PDF. No estimators, math, weighting, SEs, or assumptions affected.

Code Quality

  • No findings. Simple, scoped change in .gitignore (.gitignore:L82-L88).

Performance

  • No findings.

Maintainability

  • No findings. Ignoring papers/ helps avoid accidental large binary commits (.gitignore:L82-L88).

Tech Debt

  • No findings.

Security

  • No findings. Removing a tracked binary reduces accidental distribution risk; no secrets exposed.

Documentation/Tests

  • No findings. No code behavior changed; tests not needed.

@igerber igerber merged commit 00b26f4 into main Feb 8, 2026
1 check passed
@igerber igerber deleted the fix/remove-papers-from-git branch February 8, 2026 19:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant