Fix non-idiomatic config and bugs found during dotfiles review#64
Merged
michaelbarton merged 1 commit intomasterfrom Feb 21, 2026
Merged
Fix non-idiomatic config and bugs found during dotfiles review#64michaelbarton merged 1 commit intomasterfrom
michaelbarton merged 1 commit intomasterfrom
Conversation
Shell configs: - Remove wrong shebang from fish config.fish - Fix LESS_TERMCAP vars from set -Ux to set -gx (avoid disk writes per shell) - Remove conflicting LC_CTYPE=C (overridden by LC_ALL) and unused DISPLAY=:0 - Add missing PYTHON_BIN to fish PATH - Fix pbcat to use `command cat` (bypass bat alias) - Fix ssh function to only redirect to mosh with exactly 1 arg - Fix wt function to avoid cd (use absolute paths with rg) - Fix JAVA_HOME to be conditional (no error if Java not installed) - Replace slow `brew --prefix` with direct binary check in bashrc - Guard bash-specific init (starship/zoxide/atuin/mise) with BASH_VERSION check to prevent double-initialization when zsh sources bashrc Git: - Merge duplicate [core] sections in gitconfig - Update sw/swb/pd aliases from checkout to switch (modern idiom) - Remove dead [ghi] section (archived tool) - Remove Cargo.lock and package-lock.json from global gitignore - Rewrite clean_branches.sh: fix duplicate shebang, protect current branch Neovim: - Update trouble.lua to Trouble v3 API (TroubleToggle -> Trouble toggle) - Remove redundant todo-comments.lua (already in LazyVim core) - Remove deprecated neodev section from .neoconf.json - Add mini-move extra for line/selection motion keybindings Ansible: - Fix fish completion symlink dest to completions/ directory - Remove broken yml/format_yml.py link (file doesn't exist) - Remove deprecated ruff-lsp from neovim venv (use ruff server) Scripts: - Fix offlineimap.py: broken format string, replace removed commands module - Fix csv_to_parquet.py: check argc before accessing sys.argv[2] - Quote variables in md2pdf.sh and md2docx.sh pandoc calls - Update mutt editor from vim -f to nvim Misc: - Fix wgetrc email to match canonical address - Remove unused Clojure/lein config https://claude.ai/code/session_01V6HuQCwkGJWWzTmUGPWhhe
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Shell configs:
command cat(bypass bat alias)brew --prefixwith direct binary check in bashrccheck to prevent double-initialization when zsh sources bashrc
Git:
Neovim:
Ansible:
Scripts:
Misc:
https://claude.ai/code/session_01V6HuQCwkGJWWzTmUGPWhhe