nixvim icon indicating copy to clipboard operation
nixvim copied to clipboard

plugins: standardise telescope extension assertions

Open MattSturgeon opened this issue 1 year ago • 1 comments

Attempt to standardise how telescope extensions are enabled throughout the project.

  • Use mkIf over optional
  • Use enabledExtensions over manually calling load_extension
  • Use assertions over warnings when checking telescope is enabled
  • Don't implicitly enable telescope

The commit message could probably be worded better, or split into multiple commits?

Some of this could be refactored into helper functions to improve maintenance?

Is the assertion for plugins.telescope.extensions.*.enable overkill? Will that assertion even run when plugins.telescope.enable = false?

MattSturgeon avatar May 24 '24 21:05 MattSturgeon

I came across an auto enable behavior in todo-comments.nvim during testing a recent plugin.

khaneliman avatar Aug 10 '24 20:08 khaneliman