Repository navigation
feat: accept -R as the gh repo flag - #23
Conversation
Match gh's short repository flag while keeping --repo.
|
No actionable comments were generated in the recent review.
|
| Layer / File(s) | Summary |
|---|---|
Accept and document the repository shorthand packages/client/src/commands/gh.ts, test/unit/gh-command.test.ts, README.md |
runGh uses -R when --repo is absent and accepts R in its unknown-option check. The test and README example use -R; the common-option descriptions are aligned. |
Priority:
Estimated code review effort: 2 (Simple) | ~10 minutes
Change: Feature
Possibly related PRs
- code-vaults/git-credential-broker#22: Adds origin-based repository resolution when
--repois omitted, which PR#23extends with the-Ralias. - code-vaults/git-credential-broker#14: Introduces
gcb ghand its--repooption, which PR#23extends with-R.
Suggested reviewers: i1stg
Merge Risk: 
Minimal · up to 10a98
-
The repository shorthand change is mergeable with no identified functional or operational risk.
Pre-merge checks: 
5
-
Full details: Docstring Coverage
-
Explanation
Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 1 functions across 2 files. (1 skipped: 1 unsupported.)
-
Finishing Touches
-
📝 Generate docstrings
-
- Commit to this branch
- Create a new PR
🧪 Generate unit tests (beta)
-
- Commit to this branch
- Create a new PR
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
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 @coderabbitai help to get the list of available commands.
i1stG
left a comment
There was a problem hiding this comment.
Reviewed 10a985d: -R is resolved as an alias of --repo through the existing parser normalization, --repo and the origin-remote fallback are unchanged, and the updated test exercises the short form end to end. Full corepack yarn check is green (294 pass); CI is green. Added a README line documenting the short form. No further findings.
|
@coderabbitai review |
✅ Action performed
|
gcb ghnow accepts-R owner/repoas the short form of--repo, matching gh own flag.--repostill works.corepack yarn check: 294 pass.Summary by CodeRabbit
-Ras a shorthand for--repowhen specifying a repository.-Rand explain the shorthand.