docs: point a commit link at the current org, not the pre-rebrand one - #1778
docs: point a commit link at the current org, not the pre-rebrand one#1778ptlnextdoor wants to merge 1 commit into
Conversation
tools/contract_checks/README.md linked a fix commit under github.com/aparavi/rocketride-server, which 404s. The same commit under rocketride-org/rocketride-server returns 200. Only occurrence repo-wide. Note: the docs audit cannot catch this class -- it resolves paths inside the repo and never fetches an external URL.
🤖 Internal: Discord sync markerAuto-managed by the Discord notification workflow. Stores the linked Discord message ID and forum thread ID. Do not edit or delete. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe README production incident link for commit ChangesIncident link update
Estimated code review effort: 1 (Trivial) | ~2 minutes Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ 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 |
tools/contract_checks/README.mdlinks the fix commit under the pre-rebrand org:Verified both just now. One-line change, and the only occurrence of an old-org URL repo-wide (
grep -rn "github.com/aparavi"returns exactly this line).Other
Aparavimentions in the tree are not rebrand residue and are deliberately untouched:nodes/src/nodes/aparavi_aql/documents the Aparavi data-governance platform, which is a real third-party system this node queries.Worth noting the docs audit in #1718 cannot catch this class — it resolves paths inside the repo and never fetches an external URL. Dead external links are exactly what a rebrand leaves behind, so that gap is worth closing separately.
Summary by CodeRabbit