You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
All notifications published by the plugin are handled using the internal utils::notify module. This module dispatches the notification to the builtin neovim API nvim_notify() function, which works as expected. But in the future, the plugin should support notification decorator plugins such as nvim-notify, which overload the builtin vim.notify() function. This behavior will not be supported by the current notification module.
Proposal
Implement a separate notification module in lua, using vim.notify() function. This would require the API functions return a notification message along with the level. This can be achieved by propagating all errors using Result objects internally, and handling them at the top-level.
The text was updated successfully, but these errors were encountered:
Context
All notifications published by the plugin are handled using the internal
utils::notify
module. This module dispatches the notification to the builtin neovim APInvim_notify()
function, which works as expected. But in the future, the plugin should support notification decorator plugins such as nvim-notify, which overload the builtinvim.notify()
function. This behavior will not be supported by the current notification module.Proposal
Implement a separate notification module in lua, using
vim.notify()
function. This would require the API functions return a notification message along with the level. This can be achieved by propagating all errors usingResult
objects internally, and handling them at the top-level.The text was updated successfully, but these errors were encountered: