Skip to content

fix: pre-release sweep — gitignore, mypy, How to Use section#10

Merged
tanaysd merged 1 commit into
mainfrom
fix/pre-release-sweep
May 24, 2026
Merged

fix: pre-release sweep — gitignore, mypy, How to Use section#10
tanaysd merged 1 commit into
mainfrom
fix/pre-release-sweep

Conversation

@tanaysd

@tanaysd tanaysd commented May 24, 2026

Copy link
Copy Markdown
Owner

Summary

Pre-release sweep before making the repo public:

  • Add *.onnx, *.faiss, *.pt, mlruns/ to .gitignore — prevents accidental commits of large training artifacts
  • Fix 3 mypy no-any-return errors in DCN cross-layer forward methods (torch tensor ops typed as Any)
  • Add "How to Use This Repo" section to README — orients new visitors on learning, running, and adapting the system

Sweep results

  • No secrets, credentials, or hardcoded paths found
  • No TODO/FIXME/HACK comments
  • No dead code or broken imports
  • All __init__.py files present
  • All 6 pyproject.toml files consistent (Python >=3.11, proper metadata)
  • Lint, format, mypy, and all 105 tests pass

Test plan

  • uv run ruff check . — passes
  • uv run ruff format --check . — passes
  • uv run mypy packages/ — passes (0 errors)
  • uv run pytest tests/ranking/test_model.py -v — 19/19 pass

🤖 Generated with Claude Code

… Use section

- Add *.onnx, *.faiss, *.pt, mlruns/ to .gitignore to prevent
  accidental commits of training artifacts
- Fix 3 mypy no-any-return errors in DCN cross-layer forward methods
- Add "How to Use This Repo" section to README for new visitors

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@tanaysd
tanaysd merged commit b73d4e3 into main May 24, 2026
0 of 2 checks passed
@tanaysd
tanaysd deleted the fix/pre-release-sweep branch May 26, 2026 07:34
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