Closed Mqxx closed 7 months ago
@Mqxx Please be specific about which part of the README, and in what way.
It looks accurate to me, and I don't want to have to spend a long time figuring out what detail you noticed -- can you please share it? 🙂
hey, "\e[27;5;32~": "luafunc:fzf_complete_force" # Ctrl+Space uses fzf to filter match completions (and supports '**' for recursive).
is not mentioned in the readme, that confused me
Ok thanks.
This was actualy not the problem... Like I mentioned in my original post these lines are diffrent.
# Default key bindings for fzf with Clink.
"\C-t": "luafunc:fzf_file" # Ctrl+T lists files recursively; choose one or multiple to insert them.
"\C-r": "luafunc:fzf_history" # Ctrl+R lists history entries; choose one to insert it.
"\M-c": "luafunc:fzf_directory" # Alt+C lists subdirectories; choose one to 'cd /d' to it.
"\M-b": "luafunc:fzf_bindings" # Alt+B lists key bindings; choose one to invoke it.
"\t": "luafunc:fzf_complete" # Tab uses fzf to filter match completions, but only when preceded by '**' (recursive).
"\e[27;5;32~": "luafunc:fzf_complete_force" # Ctrl+Space uses fzf to filter match completions (and supports '**' for recursive).
# Default key bindings for fzf with Clink.
"\C-t": "luafunc:fzf_file" # Ctrl+T lists files recursively; choose one or multiple to insert them.
"\C-r": "luafunc:fzf_history" # Ctrl+R lists history entries; choose one to insert it.
"\M-c": "luafunc:fzf_directory" # Alt+C lists subdirectories; choose one to 'cd /d' to it.
"\M-b": "luafunc:fzf_bindings" # Alt+B lists key bindings; choose one to invoke it.
"\t": "luafunc:fzf_complete" # Tab uses fzf to filter match completions.
"\e[27;5;32~": "luafunc:fzf_complete" # Ctrl+Space uses fzf to filter match completions.
The function name (luafunc:fzf_complete -> luafunc:fzf_complete_force
) is diffrent.
@Mqxx Please be specific about which part of the README, and in what way.
It looks accurate to me, and I don't want to have to spend a long time figuring out what detail you noticed -- can you please share it? 🙂
@chrisant996 In my original post I have marked the lines that are diffrent.
@chrisant996 In my original post I have marked the lines that are diffrent.
Yes, that caused the confusion:
6 lines were listed from the .lua file, not the README.md file. It didn't mention which lines in the README.md file were expected to be updated. So I had to make a guess. I looked for the first lines in the README.md file that referred to the key bindings.
The first 4 lines out of the 6 are correct to be different, when comparing to the first text in the README.md about key bindings, because of differences between .md and .lua file syntax. After confirming that the first 4 lines were actually correct, I stopped spending time trying to figure out what to compare against. But now it's clear you had been looking at other lines in the README.md file.
After the follow-up reply, once I noticed one difference, I figured probably only one difference was being reported. But it turns out there were 5 specific subtle differences -- 2 in the first place in the README.md that talks about key bindings, and 3 in the second place.
It helps very much when reports describe what went wrong. For example, "I was confused because the wrong command name is listed". That would let me know exactly what the problem is and save a lot of time and confusion, but "6 lines are different" doesn't end up being very clear, especially when it cites lines from a different file than the file with the problem.
(Also, making a PR can be another clear way to show specifically what's wrong.)
Your README is not up to date with the lua script...
https://github.com/chrisant996/clink-fzf/blob/main/fzf.lua#L24-L29