Improve hosted instances docs#2196
Conversation
|
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThis PR updates the ChangesHosted Instances Documentation Updates
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~5 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 ESLint
ESLint skipped: no ESLint configuration detected in root package.json. To enable, add 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.
Pull request overview
Updates the hosted instances documentation to keep “Subgraph compatibility” descriptions more general (not tied to specific plugins) and to improve navigation by cross-linking to relevant integration and services docs.
Changes:
- Removed specific plugin-name examples from the “API-level Subgraph Compatibility” blurb while keeping the core compatibility caveat.
- Replaced Markdown
**NOTE:**insideset:htmlstrings with proper HTML emphasis to render correctly. - Added internal cross-links to Omnigraph, enssdk, enskit, and Services docs for easier discovery.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Greptile SummaryThis PR makes targeted improvements to the hosted instances documentation page. It removes specific plugin name references (Basenames, Lineanames, 3DNS) from the API-level subgraph compatibility description to keep it applicable across multiple hosted instances, normalizes
Confidence Score: 5/5This is a documentation-only change with no runtime code impact; all internal cross-links resolve to existing files. The change touches a single MDX file, makes copy/formatting improvements, and adds cross-links whose target pages were verified to exist in the repository. The NOTE: → NOTE: fix is actually a correctness improvement since these strings are rendered via Astro's set:html and markdown syntax would not be processed there. No files require special attention. Important Files Changed
Reviews (4): Last reviewed commit: "Merge branch 'main' into docs/hosted-ins..." | Re-trigger Greptile |
lightwalker-eth
left a comment
There was a problem hiding this comment.
@tk-o Thanks for updates 👍
Lite PR
Tip: Review docs on the ENSNode PR process
Summary
Why
Testing
Notes for Reviewer (Optional)
/docs/hosted-instancespage #2173Pre-Review Checklist (Blocking)