Adds install card to CLI introduction and Reorders nav#58
Conversation
|
Caution Review failedThe pull request is closed. WalkthroughThis PR reorganizes documentation navigation and updates related pages. In docs/docs.json, the CLI tab switches from a flat pages list to grouped “CLI Reference” and “Commands,” removes cli/introduction, updates Getting Started to use installation (no cli/ prefix), revises Guides (replacing guides/overview, adding local-database-setup and aws/environment-management), and renames SDK groups to “SDK Reference” and “Languages.” Page updates: docs/cli.mdx adds an “Install Suga CLI” card linking to /installation. docs/guides.mdx, docs/guides/add-suga.mdx, and docs/quickstart.mdx update links from /cli/installation to /installation and adjust minor formatting. Possibly related PRs
Suggested labels
Suggested reviewers
📜 Recent review detailsConfiguration used: CodeRabbit UI Review profile: CHILL Plan: Pro 📒 Files selected for processing (5)
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.
Actionable comments posted: 0
🧹 Nitpick comments (2)
docs/cli/introduction.mdx (1)
10-12: Install card looks good; please verify Card props and align style.
- Ensure Mintlify’s Card supports
icon="download"and thehorizontalprop in your theme; otherwise build may fail. If unsupported, drophorizontal.- Minor: other cards omit ending periods—match style for consistency.
-<Card title="Install Suga CLI" icon="download" href="/cli/installation" horizontal> - Install the Suga CLI on your system to get started. +<Card title="Install Suga CLI" icon="download" href="/cli/installation"> + Install the Suga CLI on your system to get started </Card>docs/docs.json (1)
43-46: Nav reorg reads well; confirms intro is outside Commands.No duplication detected with Getting Started. Consider whether “Using the CLI” should be “Overview” to mirror page title, but optional.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (2)
docs/cli/introduction.mdx(1 hunks)docs/docs.json(1 hunks)
🧰 Additional context used
🧠 Learnings (1)
📚 Learning: 2025-08-28T06:12:17.266Z
Learnt from: davemooreuws
PR: nitrictech/suga#29
File: docs/guides/terraform-backend-config.mdx:0-0
Timestamp: 2025-08-28T06:12:17.266Z
Learning: The Suga project uses Mintlify for documentation. Mintlify provides built-in MDX components like <Steps> and <Step> that don't need to be defined or imported in the repository - they're available by default in Mintlify's documentation platform.
Applied to files:
docs/cli/introduction.mdx
7f231e3 to
6c26ebd
Compare
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (1)
docs/docs.json (1)
24-24: Add redirect for/cli/installationin docs/docs.json@@ docs/docs.json "navigation": { @@ } }, + "redirects": [ + { "from": "/cli/installation", "to": "/installation" } + ],No occurrences of
/cli/installationwere found elsewhere; this redirect ensures old links won’t 404.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (5)
docs/cli.mdx(2 hunks)docs/docs.json(2 hunks)docs/guides.mdx(3 hunks)docs/guides/add-suga.mdx(6 hunks)docs/quickstart.mdx(12 hunks)
✅ Files skipped from review due to trivial changes (4)
- docs/quickstart.mdx
- docs/guides.mdx
- docs/cli.mdx
- docs/guides/add-suga.mdx
🔇 Additional comments (4)
docs/docs.json (4)
73-73: Rename to “SDK Reference” approved.
Label change is clear and non-breaking for routes.
29-35: Guides index exists and all “guides/overview” references removed
docs/guides.mdx is present and no lingeringguides/overviewlinks were found.
42-65: Ensure CLI docs and intro redirectVerify that every slug under the “Commands” group in
docs/docs.jsonhas a correspondingdocs/<slug>.mdxor.md, and add this to the redirects section:{ "from": "/cli/introduction", "to": "/cli" }
77-95: Confirm icon identifiers
Icons are referenced at docs/docs.json lines 81, 86, and 91. Verify your Mintlify theme supports “python”, “node-js”, and “golang” (or update to the correct names).
6c26ebd to
ac8da77
Compare
|
🎉 This PR is included in version 0.0.6 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Refactors the CLI documentation structure for better organization and discoverability.
cli.mdxfile.docs/guides/overview.mdxtodocs/guides.mdx.