Skip to content

Docker: Updated project.toml#52

Merged
AkhileshNegi merged 1 commit intomainfrom
enhancement/docker-cleanups
Feb 17, 2026
Merged

Docker: Updated project.toml#52
AkhileshNegi merged 1 commit intomainfrom
enhancement/docker-cleanups

Conversation

@AkhileshNegi
Copy link
Copy Markdown
Collaborator

@AkhileshNegi AkhileshNegi commented Feb 17, 2026

Summary

Target issue is #53

Checklist

Before submitting a pull request, please ensure that you mark these task.

  • Ran fastapi run --reload app/main.py or docker compose up in the repository root and test.
  • If you've fixed a bug or added code that is tested and has test cases.

@AkhileshNegi AkhileshNegi self-assigned this Feb 17, 2026
@AkhileshNegi AkhileshNegi added enhancement New feature or request bug Something isn't working and removed enhancement New feature or request labels Feb 17, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Feb 17, 2026

📝 Walkthrough

Walkthrough

Two dependencies—transformers and torch—were removed from the project's dependency specification in backend/pyproject.toml. No functional code changes are introduced; only the dependency list is reduced.

Changes

Cohort / File(s) Summary
Dependency Management
backend/pyproject.toml
Removed transformers and torch dependencies from project requirements.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Suggested reviewers

  • rkritika1508

Poem

🐰 Lighter loads, faster hops!
Two heavy weights we bid adieu,
The codebase springs with nimble joy,
Transformers and torch—sayonara, you!
Our dependencies lean, our hearts content.

🚥 Pre-merge checks | ✅ 3 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Title check ⚠️ Warning The title mentions updating 'project.toml' but the actual change is in 'backend/pyproject.toml', and it removes dependencies rather than performing a general update. The 'Docker:' prefix is misleading as the changes are not Docker-specific. Consider a more accurate title such as 'Remove transformers and torch dependencies from backend' to clearly reflect the actual changes made.
✅ Passed checks (3 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Merge Conflict Detection ✅ Passed ✅ No merge conflicts detected when merging into main
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch enhancement/docker-cleanups

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 and usage tips.

@AkhileshNegi AkhileshNegi linked an issue Feb 17, 2026 that may be closed by this pull request
@AkhileshNegi AkhileshNegi enabled auto-merge (squash) February 17, 2026 05:56
@AkhileshNegi AkhileshNegi merged commit 7249820 into main Feb 17, 2026
2 checks passed
@AkhileshNegi AkhileshNegi deleted the enhancement/docker-cleanups branch February 17, 2026 06:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Docker: Dependency cleanup

2 participants