andreyorst / plug.kak

Plugin manager for Kakoune
MIT License
197 stars 18 forks source link

Handle config block errors #120

Closed cole-h closed 3 years ago

cole-h commented 3 years ago

Prior to this commit, having any invalid configuration would cause plug to halt after encountering an error. If this happened in an early-enough plugin, it is possible to have no plugins actually load due to this.

This commit implements an info window that tells the user which plugin the config error originated from, as well as what the actual error was, when Kakoune opens.


Closes https://github.com/andreyorst/plug.kak/issues/118.

Example:

image

cole-h commented 3 years ago

I'll squash this back to 1 commit after you're satisfied.

andreyorst commented 3 years ago

Thanks for the changes! Looks good to me, I can squash myself if everything is ready, or you can do it if you have something else in mind for this PR.

cole-h commented 3 years ago

Nothing else in mind, I just like squashing stuff myself, because then I can be sure it will actually get squashed :sweat_smile: I've had PRs in the past with fixups get accidentally merged without them being squashed.

Thanks!