You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
check that /bin/nano exists before trying to spawn a subprocess
or better, search for nano in PATH instead of using a hardcoded path
use a universally-included editor like vi as default
if the goal was to use a more beginner-friendly editor, then perhaps resort to vim only if nano is not available
suggest using sudo -E howdy [...] in the documentation, as otherwise the command would run in a non-interactive shell (i.e. without sourcing the initialization files), and no environment variable is carried over (including EDITOR) unless explicitly preserved
The text was updated successfully, but these errors were encountered:
aleeraser
changed the title
Better EDITOR handling
Better $EDITOR handling
Jan 9, 2025
The current EDITOR handling is a bit lacking.
howdy/howdy/src/cli/config.py
Lines 14 to 20 in aa75c76
Improvements:
/bin/nano
exists before trying to spawn a subprocessnano
in PATH instead of using a hardcoded pathvi
as defaultvim
only ifnano
is not availablesudo -E howdy [...]
in the documentation, as otherwise the command would run in a non-interactive shell (i.e. without sourcing the initialization files), and no environment variable is carried over (including EDITOR) unless explicitly preservedThe text was updated successfully, but these errors were encountered: