doy / rbw

unofficial bitwarden cli
https://git.tozt.net/rbw
Other
620 stars 83 forks source link

Please add support for VISUAL environment variable and only fall back to EDITOR if the former is either empty or not set #43

Closed rjc closed 3 years ago

rjc commented 3 years ago

Hello,

I use EDITOR as well as VISUAL environment variables:

EDITOR=ed
VISUAL=vi

Both have their uses and mean different things. Most command line tools use VISUAL and only fall back to EDITOR if the former is either empty or not set, in the following order: VISUAL -> EDITOR -> vi, i.e.: ${VISUAL:-${EDITOR:-vi}}.

It would be great if a visual editor, i.e vi(1), could be used while editing entries using rbw edit ... lastpass-cli had the same issue a while back ;^)

doy commented 3 years ago

fixed in 7e52936

rjc commented 3 years ago

Thank you, much obliged! :^)