Skip to content

Test with the oldest supported Vim and Neovim - #11

Merged
leafOfTree merged 1 commit into
masterfrom
test-oldest-versions
Oct 20, 2026
Merged

leafOfTree merged 1 commit into
masterfrom
test-oldest-versions

Conversation

@leafOfTree

Copy link
Copy Markdown
Owner

Require Vim 8.2.3591+ or Neovim 0.5.1+, and test them in CI.

  • Vim 8.2.3591 adds the WinClosed event used by the commit window. Older Vim fails with E216 when committing
  • Neovim 0.5.0 ignores jobstart() stdin: 'null', so rg reads the pipe and background find in files never ends

Test changes so the suite runs on old editors (plugin code unchanged):

  • Without getscriptinfo() script variables, load the plugin from a copy where each script ends with an s: accessor
  • Find script ids by :scriptnames
  • Write binary fixtures without blobs
  • Backdate fixtures before replace in files where file times are compared in seconds
  • New test: commit from the message window
Editor Versions Result (118 tests)
Vim 8.2.3591, 8.2.3995, 9.0, 9.1, 9.2.1209 ✔
Vim 8.2.3590 and older ✘ commit window (E216)
Neovim 0.5.1, 0.6 – 0.11.6, 0.12.6 ✔
Neovim 0.5.0 ✘ background grep with rg hangs

Require Vim 8.2.3591+ (WinClosed for the commit window) or Neovim 0.5.1+
(0.5.0 ignores jobstart() 'stdin', so background search with rg never
ends), and run the tests with them in CI.

- Load the plugin from a copy with an s: accessor on editors that can't
  read script variables by getscriptinfo()
- Find script ids by :scriptnames without getscriptinfo()
- Write binary fixtures without blobs
- Backdate fixtures before replacing in files on editors that compare
  file times in seconds
- Test committing from the message window
@leafOfTree
leafOfTree force-pushed the test-oldest-versions branch from 4494ca4 to 5863cf0 Compare October 20, 2026 06:57
@leafOfTree
leafOfTree merged commit 9eaff2c into master Oct 20, 2026
16 checks passed
@leafOfTree
leafOfTree deleted the test-oldest-versions branch October 20, 2026 06:58
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.

1 participant