fix: do not use the deprecated API vim.tbl_add_reverse_lookup

rcarriga / nvim-notify

A fancy, configurable, notification manager for NeoVim

MIT License

3.03k stars 81 forks source link

fix: do not use the deprecated API vim.tbl_add_reverse_lookup #270

Closed wookayin closed 4 months ago

wookayin commented 4 months ago

vim.tbl_add_reverse_lookup has been deprecated since NVIM 0.10. Deprecations warnings will be raised since 0.11-dev.

tan-wei commented 4 months ago

Hmm, manual implement vim.tbl_add_reverse_lookup, sounds a little bit wired. However, no alternative API is provided. So seems OK for this PR, both 0.10.0 and nightly will support this.

wookayin commented 4 months ago

I don't think it is not weird. It is actually what is suggested.

tan-wei commented 4 months ago

Got it. I find that there is no alternative for the deprecated one.

rcarriga commented 4 months ago

LGTM thanks for the PR as usual! :smile: