Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Print last directory and selections to stdout instead of file #1397

Closed
wants to merge 6 commits into from
Closed

Print last directory and selections to stdout instead of file #1397

wants to merge 6 commits into from

Conversation

joelim-work
Copy link
Collaborator

@joelim-work joelim-work commented Aug 26, 2023

Changes:

  • -last-dir-path and -selection-path options are now replaced with -last-dir and -selection, which print the information to stdout instead of a file. This is a breaking change.
  • Output from running shell commands is now connected to stderr instead of stdout if either of the above options are provided.
  • Shell completions are updated.
  • The various implementations of lfcd are updated to consume the output of lf directly instead of having to create and read from a temp file.
  • The lf.vim plugin is also updated to similarly read directly from lf instead of a temp file.

I have done some testing on each of these setups, although I would greatly appreciate it if others helped to test as well, as the testing surface area is quite large and I may have missed something. For this reason I am also marking this PR as a draft.

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.

lfcd improvement proposal
1 participant