Skip to content

chore: remove recycling_bin and stray configs, tidy gitignore - #73

Merged
tomtranjr merged 1 commit into
mainfrom
chore/repo-cleanup
Sep 4, 2026
Merged

chore: remove recycling_bin and stray configs, tidy gitignore#73
tomtranjr merged 1 commit into
mainfrom
chore/repo-cleanup

Conversation

@tomtranjr

@tomtranjr tomtranjr commented Sep 4, 2026

Copy link
Copy Markdown
Member

summary

  • delete recycling_bin/ (29 tracked files, 2.7M): old deploy guides, sample configs, notebooks, a PDF, and a stray mlflow model artifact that nothing in the codebase references
  • untrack config.yaml: it held a real, personal GCP deployment config (live project_id, bucket names) that was already added to .gitignore but never removed from the tree; local copy is kept, just no longer in git
  • trim 6 dead .gitignore lines matching nothing in the current tree, including a broken --source_format=PARQUET entry that isn't a valid ignore pattern
  • add .pytest_cache/ and .ruff_cache/ to .gitignore so they can't get accidentally tracked

test plan

  • git status clean, only intended paths removed/untracked
  • confirm nothing external depended on the tracked config.yaml or recycling_bin/ contents

related

Summary by CodeRabbit

  • Documentation

    • Removed legacy deployment guides, project plans, demo instructions, and example configuration documentation.
  • Chores

    • Cleaned up ignore rules for local caches and MLflow artifacts.
    • Removed obsolete deployment configurations, local orchestration files, sample model artifacts, notebooks, and demo scripts.
    • Removed the bundled local model training and registration assets.

recycling_bin/ was a parked trash folder (old deploy guides, sample
configs, notebooks, a stray mlflow model artifact) nothing references.
config.yaml was a real, personal GCP deployment config that leaked
into git before it got added to .gitignore; untracked it but kept the
local file. Also dropped dead .gitignore lines that matched nothing in
the tree and closed the gap that let .pytest_cache/.ruff_cache get
accidentally tracked.
@coderabbitai

coderabbitai Bot commented Sep 4, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Team

Run ID: 35c48b8b-3d0f-4283-9152-197067b92fe8

📥 Commits

Reviewing files that changed from the base of the PR and between 29f4b63 and 5a550dc.

⛔ Files ignored due to path filters (3)
  • recycling_bin/MLOps project plan.pdf is excluded by !**/*.pdf
  • recycling_bin/data/house_data.parquet is excluded by !**/*.parquet
  • recycling_bin/local/mlflow-artifacts/1/models/m-342f111ca8514da183f93e043676b2e1/artifacts/model.pkl is excluded by !**/*.pkl
📒 Files selected for processing (28)
  • .gitignore
  • config.yaml
  • recycling_bin/AUTO_TEARDOWN.md
  • recycling_bin/DEPLOYMENT_GUIDE.md
  • recycling_bin/GCP_VM_Deployment.md
  • recycling_bin/GKE_DEPLOYMENT_GUIDE.md
  • recycling_bin/PROJECT_PLAN.md
  • recycling_bin/README_vm_FEAST.md
  • recycling_bin/config/deployml-489218.yaml
  • recycling_bin/config/gcp-cloud-vm-sample.yaml
  • recycling_bin/config/gcp-gke-sample.yaml
  • recycling_bin/config/gcp-sample.yaml
  • recycling_bin/config/simple-local.yaml
  • recycling_bin/gcp-cloud-vm.md
  • recycling_bin/gcp.md
  • recycling_bin/gke-deploy-test.yaml
  • recycling_bin/gke-deployment.md
  • recycling_bin/local/docker-compose.yml
  • recycling_bin/local/mlflow-artifacts/1/models/m-342f111ca8514da183f93e043676b2e1/artifacts/MLmodel
  • recycling_bin/local/mlflow-artifacts/1/models/m-342f111ca8514da183f93e043676b2e1/artifacts/conda.yaml
  • recycling_bin/local/mlflow-artifacts/1/models/m-342f111ca8514da183f93e043676b2e1/artifacts/python_env.yaml
  • recycling_bin/local/mlflow-artifacts/1/models/m-342f111ca8514da183f93e043676b2e1/artifacts/registered_model_meta
  • recycling_bin/local/mlflow-artifacts/1/models/m-342f111ca8514da183f93e043676b2e1/artifacts/requirements.txt
  • recycling_bin/local/seed_model.py
  • recycling_bin/minikube.md
  • recycling_bin/notebooks/model_registration.ipynb
  • recycling_bin/notebooks/notebook_cli_demo.ipynb
  • recycling_bin/notebooks/sandbox.ipynb
💤 Files with no reviewable changes (27)
  • recycling_bin/local/mlflow-artifacts/1/models/m-342f111ca8514da183f93e043676b2e1/artifacts/requirements.txt
  • recycling_bin/gcp-cloud-vm.md
  • recycling_bin/gke-deployment.md
  • recycling_bin/config/deployml-489218.yaml
  • recycling_bin/minikube.md
  • recycling_bin/notebooks/notebook_cli_demo.ipynb
  • recycling_bin/DEPLOYMENT_GUIDE.md
  • recycling_bin/PROJECT_PLAN.md
  • recycling_bin/config/gcp-sample.yaml
  • recycling_bin/local/mlflow-artifacts/1/models/m-342f111ca8514da183f93e043676b2e1/artifacts/MLmodel
  • recycling_bin/config/gcp-gke-sample.yaml
  • recycling_bin/config/simple-local.yaml
  • recycling_bin/notebooks/model_registration.ipynb
  • recycling_bin/local/mlflow-artifacts/1/models/m-342f111ca8514da183f93e043676b2e1/artifacts/conda.yaml
  • recycling_bin/gcp.md
  • recycling_bin/config/gcp-cloud-vm-sample.yaml
  • recycling_bin/local/mlflow-artifacts/1/models/m-342f111ca8514da183f93e043676b2e1/artifacts/python_env.yaml
  • recycling_bin/GCP_VM_Deployment.md
  • recycling_bin/gke-deploy-test.yaml
  • recycling_bin/notebooks/sandbox.ipynb
  • recycling_bin/README_vm_FEAST.md
  • recycling_bin/AUTO_TEARDOWN.md
  • recycling_bin/local/mlflow-artifacts/1/models/m-342f111ca8514da183f93e043676b2e1/artifacts/registered_model_meta
  • recycling_bin/local/docker-compose.yml
  • config.yaml
  • recycling_bin/local/seed_model.py
  • recycling_bin/GKE_DEPLOYMENT_GUIDE.md

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

The change removes archived deployment configurations, guides, notebooks, scripts, and local MLflow artifacts. It also updates .gitignore for test caches and local MLflow artifacts.

Changes

Repository cleanup

Layer / File(s) Summary
Remove local demo assets and archived materials
recycling_bin/...
Archived deployment guides, configurations, notebooks, project documentation, local MLflow metadata, and the model-seeding script are deleted.
Update ignored local files
.gitignore
The file ignores pytest and Ruff caches and local MLflow artifacts. Obsolete local paths and stray entries are removed.

Estimated code review effort: 2 (Simple) | ~10 minutes

Merge Risk: ⚪ Minimal · up to 5a550

This removes obsolete local deployment artifacts and ignores generated caches without an identified runtime or deployment impact. The change is ready to merge.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly summarizes the main changes: removing obsolete recycling_bin and configuration files, and updating .gitignore.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/repo-cleanup

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@tomtranjr tomtranjr self-assigned this Sep 4, 2026
@tomtranjr
tomtranjr merged commit 852e76e into main Sep 4, 2026
3 checks passed
@tomtranjr
tomtranjr deleted the chore/repo-cleanup branch September 4, 2026 18:17
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