-
Notifications
You must be signed in to change notification settings - Fork 3
Docs : guide deploy-on-clever #68
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 latest updates on your projects. Learn more about Vercel for GitHub.
|
WalkthroughAdds a new MDX documentation guide describing how to deploy a Start UI v3 project to Clever Cloud, covering app creation, environment variable configuration (template + placeholders), and deployment commands using the clever-tools CLI. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes
Poem
Pre-merge checks and finishing touches✅ Passed checks (3 passed)
✨ Finishing touches🧪 Generate unit tests (beta)
📜 Recent review detailsConfiguration used: CodeRabbit UI Review profile: CHILL Plan: Pro ⛔ Files ignored due to path filters (4)
📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
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: 1
🧹 Nitpick comments (5)
src/content/docs/guides/deploy.mdx (5)
31-31: Clarify step title formatting.The instruction text mixed into the step title (
*Replace all variables as needed to match your project (including your domain)*) should be moved to the step description for better readability:-2. Configure environment variables *Replace all variables as needed to match your project (including your domain)* +2. Configure environment variables + + *Replace all variables as needed to match your project (including your domain)*
40-40: Standardize placeholder naming convention.Line 40 uses
"REPLACE ME"(with space) while line 49 uses"REPLACE_ME"(with underscore). Choose one convention consistently:- DATABASE_URL="REPLACE_ME" + DATABASE_URL="REPLACE ME"This ensures users can easily identify all placeholders that need replacement.
Also applies to: 49-49
51-51: Clarify EMAIL_SERVER placeholder.Line 51's
EMAIL_SERVER="smtp://your-smtp-server"is not clearly marked as a placeholder requiring replacement (unlike other variables usingREPLACE ME). Consider standardizing:- EMAIL_SERVER="smtp://your-smtp-server" + EMAIL_SERVER="REPLACE ME"or add a comment to clarify it needs configuration.
39-59: Consider documenting environment variable purposes.The environment variable template lacks descriptions for less obvious variables (e.g.,
VITE_ENV_COLOR,VITE_ENV_EMOJI,VITE_ENV_NAME). A brief comment explaining their purpose would improve usability.Would you like me to add inline documentation explaining the purpose of key environment variables?
61-72: Consider adding troubleshooting and Node version guidance.The guide would benefit from:
- Node.js version requirement for Clever Cloud deployment
- Common troubleshooting steps if deployment fails
- How to access the deployed app after successful deployment
- How to view real-time logs during deployment
These additions would make the guide more self-contained and reduce support burden.
Would you like me to expand the guide with troubleshooting sections and additional deployment context?
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (3)
src/assets/guides/deploy-on-clever/deploy-on-clever-1.pngis excluded by!**/*.pngsrc/assets/guides/deploy-on-clever/deploy-on-clever-2.pngis excluded by!**/*.pngsrc/assets/guides/deploy-on-clever/deploy-on-clever-3.pngis excluded by!**/*.png
📒 Files selected for processing (1)
src/content/docs/guides/deploy.mdx(1 hunks)
🔇 Additional comments (1)
src/content/docs/guides/deploy.mdx (1)
20-20: Image assets verified—all files exist.All three image assets referenced in deploy.mdx are present at the correct paths:
- deploy-on-clever-1.png ✓
- deploy-on-clever-2.png ✓
- deploy-on-clever-3.png ✓
ecfe7ce to
de7befc
Compare
de7befc to
0729cbd
Compare
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: 2
🧹 Nitpick comments (1)
src/content/docs/guides/deploy-on-clever-cloud.mdx (1)
29-29: Clarify step title formatting.The asterisks create markdown emphasis, which may be unintentional. Consider reformatting for clarity—either as separate instructional text below the step or in a callout box to emphasize the requirement.
Current (renders as italics):
2. Configure environment variables *Replace all variables as needed to match your project (including your domain)*Suggested alternative:
2. Configure environment variables **⚠️ Replace all variables as needed to match your project (including your domain)**
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
src/content/docs/guides/deploy-on-clever-cloud.mdx(1 hunks)
🔇 Additional comments (1)
src/content/docs/guides/deploy-on-clever-cloud.mdx (1)
1-79: Well-structured deployment guide with good instructional flow.The guide effectively breaks down the Clever Cloud deployment process into clear, actionable steps with helpful visuals and a comprehensive environment variable template. The past issue with duplicate image alt text has been resolved—each image now has a distinct reference (deploy-on-clever-1, deploy-on-clever-2, deploy-on-clever-3).

Description
This PR aim to open guides to deploy the app, and starting to add a clever cloud guide to it
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.