Repository navigation
feat: add {random} / {random:N} project name placeholder's - #2431
mynameistito wants to merge 4 commits into
Conversation
{random} / {random:N} project name placeholder's
|
Thanks @mynameistito! Greptile is at 4/5 here. We only merge once Greptile reaches 5/5, so please address its findings in the summary above (or reply to them if you think one is wrong), then comment |
|
@greptile |
|
@greptileai review |
Hey will do! I believe my last fixed fixed up the greptile issue! |
|
@mynameistito is attempting to deploy a commit to the Cap Software Inc Team on Vercel. A member of the Team first needs to authorize it. |
|
@greptileai review this |
|
All 5/5 now @richiemcilroy |
Closes #2430.
Adds
{random}(10 lowercase hexadecimal characters) and{random:N}(1–32 characters) to project-name templates in both desktop recording paths. Documents the placeholders in settings and the GPUI preview behavior.Validation: Biome and Rust formatting pass. Focused Rust tests could not run in this environment: the GPUI crate depends on a missing local
zed-capcheckout, and the Tauri crate's pinnedcidresource is unavailable offline.

Confidence Score: 5/5
The PR appears safe to merge, with both previous findings fixed and no new actionable issues found.Summary
Adds
{random}and{random:N}to project-name templates in both desktop recording paths.Reviews (3) · Last reviewed commit: "fix: import chrono timezone in project n..." · Reviewed by Greptile