word diff regression #191

Closed

phanen wants to merge 2 commits from word_diff_regression into main

phanen commented

2026-03-13 12:02:43 +00:00

chore: word_diff regresion
test: diff.has_vscode() fail

word_diff seems missing again (https://github.com/neovim/neovim/issues/31151)

Not sure how to fix the test, so take this as a bug report.

- **chore: word_diff regresion** - **test: diff.has_vscode() fail** word_diff seems missing again (https://github.com/neovim/neovim/issues/31151) Not sure how to fix the test, so take this as a bug report.

phanen commented

2026-03-13 12:13:33 +00:00

There're a few other choice for fine grid extmark test:

screenattr() also used in mini.test, but can be flaky to nvim version bump
ext_linegrid ui client a bit complex here, since this require subprocesses github.com/neovim/neovim@d0c699ec7b/test/functional/ui/screen.lua
nvim__inspect_cell() non-stable, for float window it need to "search" for the correct "grid"

There're a few other choice for fine grid extmark test: * `screenattr()` also used in mini.test, but can be flaky to nvim version bump * `ext_linegrid` ui client a bit complex here, since this require subprocesses https://github.com/neovim/neovim/blob/d0c699ec7b0fe6f942536b8241e35575f3b9d5ff/test/functional/ui/screen.lua * `nvim__inspect_cell()` non-stable, for float window it need to "search" for the correct "grid"

barrettruth commented

2026-03-14 00:58:36 +00:00

damnit. sorry about this. going to see what i can do.

barrettruth commented

2026-03-14 01:21:08 +00:00

#192