feat(installer): add viz to repo menu (option 9)#18
Conversation
The viz repo (3D visualization — Three.js/Cesium web viewers + Unity .NET) is public, documented in dev/README.md's ecosystem table, and missing from both install.sh and install.ps1's REPO menu. Adding it closes the drift: - install.sh: VALID_REPOS gains `viz`; menu prompt extends to 1-9; case arm `9) REPO="viz"`; print_repo_info adds a viz section describing the TypeScript web viewers + Unity .NET 9 solution file. - install.ps1: $ValidRepos gains 'viz'; menu shows option 9; switch maps '9' -> 'viz'; Show-RepoInfo gains a matching viz branch. Ardupilot is public but a fork of ArduPilot/ardupilot — not added to the installer (it's not a ResQ dev repo).
|
Warning Rate limit exceeded
Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 13 minutes and 53 seconds. ⌛ 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. ✨ 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 |
There was a problem hiding this comment.
Code Review
This pull request adds a new repository option named 'viz' for 3D visualization to both the PowerShell (install.ps1) and Shell (install.sh) installation scripts. The changes include updating the valid repository lists, adding a new menu selection (option 9), and providing descriptive information for the new repository. I have no feedback to provide.
Summary
The `viz` repo (3D visualization — Three.js/Cesium web viewers + Unity .NET) is public, listed in `dev/README.md`'s ecosystem table, but was missing from both installers. Drift closed.
`ardupilot` is public but a fork of `ArduPilot/ardupilot` — not added (it's not a ResQ dev repo).
Test plan
Follow-up (not in this PR)
If the org adds more repos frequently, switch the hardcoded menu to a runtime `gh api /orgs/resq-software/repos` fetch with the hardcoded list as fallback. Worth doing if this breaks 10+ entries; fine hardcoded for now.
🤖 Generated with Claude Code