Skip to content

Comments

fix: rename gtr.fish to git-gtr.fish#71

Merged
helizaga merged 1 commit intocoderabbitai:mainfrom
scarf005:fix/fish-completion-gtr-to-git-gtr
Jan 5, 2026
Merged

fix: rename gtr.fish to git-gtr.fish#71
helizaga merged 1 commit intocoderabbitai:mainfrom
scarf005:fix/fish-completion-gtr-to-git-gtr

Conversation

@scarf005
Copy link
Contributor

@scarf005 scarf005 commented Jan 3, 2026

fish autocompletion didn't work because installed binary name is git-gtr, not gtr.

Summary by CodeRabbit

  • Documentation
    • Updated installation guides and documentation to reference the renamed Fish shell completion file (now git-gtr.fish) so setup instructions point to the correct completion name.

✏️ Tip: You can customize this high-level summary in your review settings.

@scarf005 scarf005 requested a review from NatoBoram as a code owner January 3, 2026 11:15
@coderabbitai
Copy link

coderabbitai bot commented Jan 3, 2026

Walkthrough

Documentation updates changing Fish shell completion filename references from gtr.fish to git-gtr.fish across README, CLAUDE.md, and the completion installation example.

Changes

Cohort / File(s) Summary
Documentation references
CLAUDE.md, README.md
Replaced references to completions/gtr.fish with completions/git-gtr.fish in narrative and examples.
Fish completion installation example
completions/git-gtr.fish
Updated symlink/installation example to use git-gtr.fish (renamed target).

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 A tiny hop to change a name,
From gtr to git-gtr the path became,
Docs refreshed and symlinks too,
A rabbit cheers — quick, neat, and true! 🥕

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title directly and clearly describes the main change: renaming the Fish shell completion file from gtr.fish to git-gtr.fish, which aligns with all file updates documented in the pull request.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Disabled knowledge base sources:

  • Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between 2ea339e and df99362.

📒 Files selected for processing (3)
  • CLAUDE.md
  • README.md
  • completions/git-gtr.fish
🚧 Files skipped from review as they are similar to previous changes (2)
  • completions/git-gtr.fish
  • README.md
🔇 Additional comments (1)
CLAUDE.md (1)

406-406: Update correctly aligns documentation with Fish completion filename change.

The reference to the Fish shell completion file has been updated to match the renamed binary name (git-gtr instead of gtr). This ensures users follow the correct completion installation path, which should resolve Fish autocompletion issues described in the PR.


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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

fish autocompletion didn't work because installed binary name is `git-gtr`, not `gtr`.

see: fish-shell/fish-shell#7075
@scarf005 scarf005 force-pushed the fix/fish-completion-gtr-to-git-gtr branch from 2ea339e to df99362 Compare January 3, 2026 11:40
@helizaga helizaga merged commit cca0bc8 into coderabbitai:main Jan 5, 2026
1 check passed
helizaga added a commit that referenced this pull request Jan 5, 2026
Aligns with the file rename in PR #71 (commit cca0bc8).

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
helizaga pushed a commit that referenced this pull request Feb 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants