-
Notifications
You must be signed in to change notification settings - Fork 50
fix: config get/unset now handle multi-value keys correctly #37
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. Weβll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
WalkthroughThe changes modify git configuration handling to support multi-value keys. The Changes
Estimated code review effortπ― 1 (Trivial) | β±οΈ ~5 minutes
Poem
Pre-merge checks and finishing touchesβ Passed checks (3 passed)
β¨ Finishing touches
π§ͺ Generate unit tests (beta)
π Recent review detailsConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Disabled knowledge base sources:
π Files selected for processing (2)
π§° Additional context usedπ Path-based instructions (7)lib/**/*.shπ CodeRabbit inference engine (.github/copilot-instructions.md)
Files:
**/*.shπ CodeRabbit inference engine (.github/copilot-instructions.md)
Files:
lib/config.shπ CodeRabbit inference engine (.github/copilot-instructions.md)
Files:
**/*.{bash,fish,sh}π CodeRabbit inference engine (.github/instructions/sh.instructions.md)
Files:
{bin/gtr,lib/**/*.sh,adapters/**/*.sh}π CodeRabbit inference engine (.github/instructions/testing.instructions.md)
Files:
lib/*.shπ CodeRabbit inference engine (CLAUDE.md)
Files:
bin/gtrπ CodeRabbit inference engine (.github/copilot-instructions.md)
Files:
π Additional comments (2)
Comment |
Pull Request
Description
Motivation
Fixes # (issue)
Type of Change
Testing
Manual Testing Checklist
Tested on:
Core functionality tested:
git gtr new <branch>- Create worktreegit gtr go <branch>- Navigate to worktreegit gtr editor <branch>- Open in editor (if applicable)git gtr ai <branch>- Start AI tool (if applicable)git gtr rm <branch>- Remove worktreegit gtr list- List worktreesgit gtr config- Configuration commands (if applicable)Test Steps
Expected behavior:
Actual behavior:
Breaking Changes
Checklist
Before submitting this PR, please check:
git gtr(production) and./bin/gtr(development)Additional Context
License Acknowledgment
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache License 2.0.
Summary by CodeRabbit
βοΈ Tip: You can customize this high-level summary in your review settings.