fix: correct shell config bugs in .zshrc and .zsh_aliases#6
fix: correct shell config bugs in .zshrc and .zsh_aliases#6tschulze wants to merge 1 commit intoCodeOpsHQ:mainfrom
Conversation
- Remove trailing 'visudo' from EDITOR and VISUAL exports in .zshrc and .zsh_aliases — value was set to "nvim visudo" instead of "nvim" - Guard fzf preview alias with a bat availability check, fall back to cat when bat is not installed - Fix missing '#' on Zoxide and Tmuxifier comment lines in .zshrc - Comment out zoxide and tmuxifier eval lines in .zshrc - Rename zsh-syntax-highlightin-tokyonight.zsh (missing 'g') - Remove duplicate _SKIP_* variable block accidentally pasted inside the nnn plugin if-block in .zsh_aliases
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (3)
📝 WalkthroughWalkthroughShell configuration files are corrected to remove accidental ChangesShell Configuration Cleanup
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ 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. Review rate limit: 0/1 reviews remaining, refill in 60 minutes.Comment |
Summary
visudofromEDITORandVISUALexports in.zshrcand.zsh_aliases— the value was being set to"nvim visudo"instead of"nvim", breaking any tool that invokes$EDITORfzfpreview alias with abatavailability check and fall back tocatwhenbatis not installed#on the Zoxide and Tmuxifier comment lines in.zshrc(bare text was being executed as commands)zoxideandtmuxifiereval lineszsh-syntax-highlightin-tokyonight.zsh→zsh-syntax-highlighting-tokyonight.zsh(missingg)_SKIP_*variable block that was accidentally pasted inside thennnpluginif-block in.zsh_aliases, causing all those variables to be reset whenevernnnis installedTest plan
.zshrcand verifyecho $EDITORreturnsnvimfzfpreview works with and withoutbatinstalledSummary by CodeRabbit
Bug Fixes
Improvements