Skip to content

feat(env): respect environment variables for editor - #235

Open
astropanic wants to merge 7 commits into
clearloop:mainfrom
astropanic:main
Open

feat(env): respect environment variables for editor#235
astropanic wants to merge 7 commits into
clearloop:mainfrom
astropanic:main

Conversation

@astropanic

@astropanic astropanic commented Aug 9, 2026

Copy link
Copy Markdown

Set up the editor based on the VISUAL and EDITOR environment variables.

Precedence:

  • VISUAL env
  • EDITOR env
  • configuration file setup
  • fallback to vim

Additionally update the code to stop clippy from complaining.

@astropanic
astropanic force-pushed the main branch 2 times, most recently from a27ea88 to d01f7ed Compare August 9, 2026 07:30
Comment thread src/config/code.rs
Comment thread src/config/code.rs
Comment thread docs/configuration.md Outdated
Comment thread docs/configuration.md Outdated
Comment thread tests/config_code.rs
@clearloop

clearloop commented Aug 9, 2026

Copy link
Copy Markdown
Owner

thanks for the contribution! could you please bump the patch version btw that once this PR getting merged, it can be published to crates-io directly!

astropanic and others added 5 commits August 10, 2026 09:18
Co-authored-by: clearloop <tianyi.gc@gmail.com>
Co-authored-by: clearloop <tianyi.gc@gmail.com>
Co-authored-by: clearloop <tianyi.gc@gmail.com>
Co-authored-by: clearloop <tianyi.gc@gmail.com>
Co-authored-by: clearloop <tianyi.gc@gmail.com>
Comment thread src/config/code.rs
Comment thread tests/config_code.rs
@clearloop

Copy link
Copy Markdown
Owner

astropanic#1 @astropanic ^ ^

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