Repository navigation
chore: remove todo-worker and todo-worker-python - #369
Conversation
Remove the Node and Python quickstart CRUD todo workers and strip their references from the README worker table and onboarding docs (sops/README, new-worker, binary-worker, deploy-modes) to avoid broken links.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (29)
💤 Files with no reviewable changes (26)
📝 WalkthroughWalkthroughRemoves the Changestodo-worker and todo-worker-python removal
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~5 minutes Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
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. Comment |
skill-check — worker0 verified, 27 skipped (no docs/).
Four for four. Nicely done. |
What
Removes the
todo-worker(Node) andtodo-worker-python(Python) quickstart CRUD todo workers.Why
Dead workers — removing them from the monorepo.
Doc cleanup
They were cited as the canonical Node/Python
imagescaffold templates, so references were stripped to avoid broken links:README.md— removed both rows from the worker tabledocs/sops/README.md— dropped Node/Python container scaffold bullets from "Typical flow"docs/sops/new-worker.md— scaffold tableimagerows now point todeploy-modes.md §Image; removed "todo-worker templates" from sources-of-truthdocs/architecture/deploy-modes.md— removed theTemplates:line under §Imagedocs/sops/binary-worker.md— droppedtodo-workerfrom the Path A exampleThe two
todo-workerhits left iniii-directory/src/functions/registry.rsare unrelated test-fixture strings (an arbitrary dependency name), not directory references.Summary by CodeRabbit