fix: address CodeRabbit nitpicks from PR #146#150
Conversation
- Patch os.path.lexists (not os.path.exists) in avatar tests to match the production guard in _download_avatar - Add skipped_symlinks counter to verify output so users see how many symlink entries were intentionally bypassed - Bump version to 7.7.5
|
Warning Rate limit exceeded
You’ve run out of usage credits. Purchase more in the billing tab. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (4)
✨ Finishing Touches🧪 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 |
|
🐳 Dev images published!
The dev/test instance will pick up these changes automatically (Portainer GitOps). To test locally: docker pull drumsergio/telegram-archive:dev
docker pull drumsergio/telegram-archive-viewer:dev |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #150 +/- ##
=======================================
Coverage 94.00% 94.01%
=======================================
Files 21 21
Lines 6139 6144 +5
=======================================
+ Hits 5771 5776 +5
Misses 368 368
🚀 New features to boost your workflow:
|
Summary
os.path.lexistsinstead ofos.path.existsin avatar tests to match the production guard in_download_avatar(tests were environment-dependent)skipped_symlinkscounter to_verify_and_redownload_mediaoutput so users see how many symlink entries were intentionally bypassed during verificationAddresses CodeRabbit nitpicks from #146.
Test plan