Conversation
WalkthroughThis pull request restructures the documentation navigation and expands language driver coverage across three key sections: adding 20+ language-specific driver cards and Software Testing to the connect guide, reorganizing handbook navigation entries, and simplifying the integrate category overview while removing redundant Software Testing content. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Possibly related PRs
Suggested labels
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ 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 |
| :glob: | ||
|
|
||
| **/index | ||
| */index |
There was a problem hiding this comment.
Perhaps move the "By category" up to the top, perhaps even enumerate them directly?
And then the long index afterwards?
There was a problem hiding this comment.
bmunkholm
left a comment
There was a problem hiding this comment.
Great step, and in itself a good valid improvement.
More needs done with the integrations as the TOC index is simply too long. But let's leave that for another PR and this focusing on dissolving the "Topic" 👍.
Optional: remove Rockset migration entirely? Likely no one is left to migrate.
|
I've merged so I can build on top of this. We can always fix some of the suggestions I had later. |
|
About
A few inches towards a recent proposal by @bmunkholm and @kneth to improve the primary navigation.
Details
Preview
-- https://cratedb-guide--543.org.readthedocs.build/
-- https://cratedb-guide--543.org.readthedocs.build/connect/testing/
References
/cc @surister, @matriv, @seut