Uh oh!
There was an error while loading. Please reload this page.
feat(cmd): add hidden flag aliases for devcontainer CLI compat - #200
Conversation
Add --override-config, --dotfiles-repository, and --remove-existing-container as hidden aliases on the up command to match the official devcontainer CLI naming conventions.
✅ Deploy Preview for devsydev canceled.
|
Warning Rate limit exceeded
To keep reviews running without waiting, you can enable usage-based add-on for your organization. This allows additional reviews beyond the hourly cap. Account admins can enable it under billing. ⌛ 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: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
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. Review rate limit: 0/1 reviews remaining, refill in 46 minutes and 41 seconds.Comment |
Uh oh!
There was an error while loading. Please reload this page.
Summary
Adds hidden cobra flag aliases on the
upcommand to match the official devcontainer CLI naming conventions, improving compatibility for users migrating from or scripting against the standard CLI.Aliases added:
--override-config→ alias for--extra-devcontainer-path(matchesdevcontainer up --override-config)--dotfiles-repository→ alias for--dotfiles(matchesdevcontainer up --dotfiles-repository)--remove-existing-container→ alias for--recreate(matchesdevcontainer up --remove-existing-container)Aliases skipped (with reasons):
--workspace-folderonup: Currently a positional arg (up [workspace-path]). Adding as a flag would conflict with positional arg semantics. Theexeccommand already uses--workspace-folderas an explicit flag.--remote-envonup: The official CLI only has--remote-envonexec(notup). Our--workspace-envonuphas different semantics (injected during workspace creation, not remote container env).--container-data-folder/--container-system-data-folder: No equivalent flags exist in our CLI.--mount: Not an official devcontainer CLI flag. The official CLI has--mount-workspace-git-rootinstead, which is semantically different from our--workspace-mount-consistency.--id-label: Already uses the same name as the official CLI — no alias needed.All aliases are hidden (don't appear in
--help) and bind to the same variable as the primary flag. Unit tests cover both value binding and hidden status for each alias.