rcarriga / nvim-notify

A fancy, configurable, notification manager for NeoVim
MIT License
3.03k stars 81 forks source link

fix: only warn for termguicolors if version minor < 10 #239

Closed seblj closed 9 months ago

seblj commented 9 months ago

Neovim now sets termguicolors default to true if the terminal supports it. However, the option is not set until VimEnter, and this means that reading the value of termguicolors will always be false if the user has not set it in their config.

After I removed the unnecessary default option after updating neovim, I always get the warning now.

Check the version as well before warning since termguicolors is now set to true by default if the terminal supports it

rcarriga commented 9 months ago

Thanks for the PR!