thenbe / neotest-playwright

A playwright adapter for neotest.
MIT License
39 stars 5 forks source link

pickers.lua throws an error for telescope installation when the option is disabled #38

Closed oedotme closed 2 months ago

oedotme commented 2 months ago

Telescope installation is optional (and experimental), but causes the adapter to fail if it's not installed.

Could it be possible to throw error only if the option is enabled?

https://github.com/thenbe/neotest-playwright/blob/f61463f5c74d53fc5975b7e086b7c6e9356d9215/src/pickers.lua#L3-L5

if not has_telescope then
  if options.experimental.telescope.enabled then error('nvim-telescope/telescope.nvim is not installed') end
  return
end
thenbe commented 2 months ago

Now, it should only show an error if experimental.telescope.enabled = true.