sheerun / vim-polyglot

A solid language pack for Vim.
5.59k stars 295 forks source link

Get scripts/build working again, make the tests pass #844

Open 00dani opened 1 year ago

00dani commented 1 year ago

I noticed there hasn't been an update to vim-polyglot in about a year, and that the test suite was failing, so I figured I'd try to fix both issues. There were some obvious issues, like calling File.exists? rather than File.exist? in the build script, but I've made some somewhat more drastic changes as well.

Most notably, all native packages aren't copied into vim-polyglot itself any more, because that causes conflicts between the version supplied by Vim and the version supplied by vim-polyglot. For example, running :setf ada with the mainline version of vim-polyglot will error, because Vim will load both its own version of the Ada plugin and vim-polyglot's version, and that plugin uses :lockvar so you get a crash.

I've also tweaked the test suite slightly to make it more robust, and I've taught generate_ftdetect to include suitable autocommands for filetypes Vim natively knows about but vim-polyglot does not, such as Factor, Hack, and Twig.

Finally I ran a build and committed the updated vim-polyglot, which now passes all its tests.