Repository navigation
fix: use dynamic HOME path instead of hardcoded /home/rizzo - #4
itsdarklikehell wants to merge 1 commit into
Conversation
- Replaced hardcoded '/home/rizzo' with os.path.expanduser('~')
- Now works for any user, not just 'rizzo'
There was a problem hiding this comment.
Sorry @itsdarklikehell, you've used your own review budget of 250,000 diff characters for the last 7 days.
You can request another review in 1 hour and 50 minutes by commenting @sourcery-ai review. Upgrade to get a review now.
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID:
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 |
Reviewer's guide (collapsed on small PRs)Reviewer's GuideThe PR removes the hardcoded File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
There was a problem hiding this comment.
Your trial has ended. Reactivate Greptile to resume code reviews.
Up to standards ✅🟢 Issues
|
| Metric | Results |
|---|---|
| Complexity | 0 |
| Duplication | 0 |
NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.
Changes
/home/rizzowithos.path.expanduser('~')Verification
python3 -m py_compile potfilesorter.py✅ Syntax OKSummary by Sourcery
Bug Fixes: