GitHub workflow multiple vim version

shadowwa commented 2 months ago

update github action to prevent warning on action run. Had to define python version to prevent another warning, I used 3.10 as it's the version of ubuntu 22.04LTS currently used by github.
allow to have several versions of vim or nvim tested (unfortunatly, neovim below 0.9.0 segfault, even when trying to install the apt version available)

lelutin commented 2 months ago

THANKS! :heart: I've been meaning to do exactly this change in the past few days but haven't reached that yet.

it's too bad for the previous versions of nvim -- the segfault was exactly my experience with it before, which is why nvim was just removed entirely from CI for a good while -- but at least we can test starting with 0.9 which is good.

do you think we could also add nvim 0.10.x to the matrix? debian unstable and ubuntu (I think? packages.ubuntu.com is not responding right now) still ship 0.9.5 but fedora has had 0.10.1 for a little while

edit: err scratch that.. I had apparently not looked correctly. nvim 0.10.1 is already in the list

lelutin commented 2 months ago

apparently this uncovers issues in epuppet ftdetect tests with vim 8.0

the type auto-discovery goes to 'mason' instead of 'epuppet'
somehow the default type of 'sh' is not there

This does not cause the bugs to exist, just uncovers it. I'll merge as-is and then open an issue for the vim 8.0 issues

rodjek / vim-puppet

GitHub workflow multiple vim version #153