Page MenuHomeSolus

Update fzf to 0.18.0 and include the vim plugin
Needs ReviewPublic

Authored by xulongwu4 on Sun, Jun 9, 4:16 PM.

Details

Reviewers
JoshStrobl
Group Reviewers
Triage Team
Summary

Update fzf to 0.18.0.
This patch also include the vim plugin, zsh auto-completion and shell key bindings.
Changelog:

  • Added placeholder expression for zero-based item index: {n} and {+n}
    • fzf --preview 'echo {n}: {}'
  • Added color option for the gutter: --color gutter:-1
  • Added --no-unicode option for drawing borders in non-Unicode, ASCII characters
  • FZF_PREVIEW_LINES and FZF_PREVIEW_COLUMNS are exported to preview process
    • fzf still overrides LINES and COLUMNS as before, but they may be reset by the default shell.
  • Bug fixes and improvements
Test Plan
  • Tested the vim plugin in vim using the FZF command
  • Tested the fzf command

Diff Detail

Repository
R864 fzf
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
xulongwu4 created this revision.Sun, Jun 9, 4:16 PM
xulongwu4 requested review of this revision.Sun, Jun 9, 4:16 PM
xulongwu4 edited the summary of this revision. (Show Details)Sun, Jun 9, 4:38 PM
JoshStrobl requested changes to this revision.Tue, Jun 11, 7:24 PM
JoshStrobl added a subscriber: JoshStrobl.
JoshStrobl added inline comments.
package.yml
20–29

To be clear, nothing wrong with your patch, but while we're updating this, let's go ahead and try to have the install calls be more aligned with other packages by consolidating the flags so instead of -D -m it's -Dm (example -Dm00755) and -d -m changes to -dm. Thanks for the patch!

This revision now requires changes to proceed.Tue, Jun 11, 7:24 PM
xulongwu4 updated this revision to Diff 15634.Thu, Jun 13, 11:40 AM

Fix the styles of the install calls.

xulongwu4 marked an inline comment as done.Thu, Jun 13, 11:41 AM
xulongwu4 updated this revision to Diff 15675.Sun, Jun 16, 2:26 PM

Rebase on master