-
-
Notifications
You must be signed in to change notification settings - Fork 317
docs(pi): fix pi-agent repository link #800
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
The link was incorrectly changed to a non-existent repository (nicobailon/pi-agent) in f86d0f3. Restore the correct link to badlogic/pi-mono.
📝 WalkthroughWalkthroughThis pull request updates documentation references across two README files, replacing repository URLs from Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. 📜 Recent review detailsConfiguration used: defaults Review profile: CHILL Plan: Pro 📒 Files selected for processing (2)
🧰 Additional context used📓 Path-based instructions (1)**/*.md📄 CodeRabbit inference engine (CLAUDE.md)
Files:
🧠 Learnings (1)📚 Learning: 2026-01-09T12:35:27.240ZApplied to files:
🔇 Additional comments (3)
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 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
apps/ccusage/README.md (1)
42-42: Unrelated change included in this PR.The addition of "and credits" to the Amp description appears unrelated to the PR objective of fixing pi-agent repository links. While this change may be valid, it should ideally be in a separate commit or PR for clarity.
Consider whether this change should be:
- Kept in this PR with an updated PR description
- Moved to a separate PR focused on documentation improvements
📜 Review details
Configuration used: defaults
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (2)
apps/ccusage/README.mdapps/pi/README.md
🧰 Additional context used
📓 Path-based instructions (1)
**/*.md
📄 CodeRabbit inference engine (CLAUDE.md)
**/*.md: In documentation pages with screenshots, always place screenshots immediately after the main heading (H1)
Use relative image paths like/screenshot.pngfor images stored in/docs/public/
Always include descriptive alt text for images in documentation for accessibility
Files:
apps/pi/README.mdapps/ccusage/README.md
🧠 Learnings (1)
📚 Learning: 2026-01-09T12:35:27.240Z
Learnt from: CR
Repo: ryoppippi/ccusage PR: 0
File: apps/amp/CLAUDE.md:0-0
Timestamp: 2026-01-09T12:35:27.240Z
Learning: Reuse shared packages ccusage/terminal and ccusage/internal wherever possible in Amp CLI
Applied to files:
apps/ccusage/README.md
🔇 Additional comments (3)
apps/ccusage/README.md (1)
38-38: URL fix is correct and consistent.The pi-agent repository link has been properly updated to
badlogic/pi-mono, matching the changes inapps/pi/README.md.apps/pi/README.md (2)
72-72: URL fix is consistent.The repository link has been correctly updated to
https://github.com/badlogic/pi-monoat both line 14 and line 72. No remaining references to the incorrect URL exist.
14-14: URL fix verified. The repository link tobadlogic/pi-monois correct and publicly accessible.
Fix broken pi-agent repository links in apps/pi/README.md.
The links were incorrectly changed from
badlogic/pi-monotonicobailon/pi-agent(non-existent) in f86d0f3.Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.