bug: Diff review doesn't skip binary files

[SevereOverfl0w]

Prerequisites

• I have searched existing issues
• I have updated to the latest version

Neovim version

NVIM v0.13.0-dev-726+ga69f2db773-Homebrew

Operating system

MacOS

Description

Pressing ]f gave me an error message: diff does not support binary files. I would hope to either skip the binary files OR do something sensible (show me "--binary" in a buffer or whatever git does).

In any case, I couldn't skip the binary file to continue my review, not even with 2]f to jump over it or similar.

Steps to reproduce

1. Modify a text file a and a binary file b and commit it
2. Diff review ++layout=split HEAD^
3. ]f

Health check

==============================================================================
diffs:                                                                      ✅

diffs.nvim ~
- ✅ OK Neovim 0.9.0+ detected
- ✅ OK vim-fugitive detected
- libvscode_diff not found (optional, using native vim.diff fallback)
- difftastic not enabled (optional, set integrations.difftastic

Minimal reproduction

it did reproduce with this config.

[barrettruth]

Thanks for taking the time to file a report.

I now log skipped files at an info-level; follow up in this thread if the fix is insufficient!

[SevereOverfl0w]

That works a treat, the log doesn't actually show up for me when skipping around with ]f but I do see it in the :messages output. I'd also be OK with it being logged when doing :Diff review ...

[barrettruth]

duh... see #420 blaze it... just fixed the skipped log "on top" of the current log.