Add optional VS Code devcontainer#477
Conversation
Deploying with
|
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs |
unraid-docs | e891d3c | Commit Preview URL Branch Preview URL |
May 13 2026, 06:07 PM |
|
Warning Rate limit exceeded
You’ve run out of usage credits. Purchase more in the billing tab. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe pull request adds VS Code dev container support to the documentation repository. It introduces a Dockerfile and devcontainer.json for a Node.js development environment, creates a Docusaurus process wrapper script with signal handling and configurable host/port binding, updates npm scripts to use the wrapper, and documents the container setup in the README. ChangesVS Code Dev Container and Docusaurus Wrapper
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Warning Review ran into problems🔥 ProblemsGit: Failed to clone repository. Please run the 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: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@scripts/run-docusaurus.mjs`:
- Around line 5-31: The script only uses process.argv[2] (commandName) and
builds a fixed args array for spawn, so any extra flags after the command (e.g.
--no-open) are dropped; update the spawn invocation that constructs the args for
the docusaurus call (the array passed to spawn in the variable child) to append
process.argv.slice(3) (the extra CLI args) to
["exec","docusaurus",commandName,"--host",host,"--port",port] so all
user-provided flags are forwarded; ensure you still use the existing
commandName, host and port variables and keep stdio: "inherit".
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Repository UI
Review profile: CHILL
Plan: Pro
Run ID: 989cefa9-9a37-4d27-a1b9-5c42381dcdc1
📒 Files selected for processing (6)
.devcontainer/Dockerfile.devcontainer/devcontainer.json.gitignoreREADME.mdpackage.jsonscripts/run-docusaurus.mjs
Before Submitting This PR, Please Ensure You Have Completed The Following:
Summary by CodeRabbit
Release Notes
New Features
Documentation
Chores