Fix: use vim.notify in packer.log

wbthomason / packer.nvim

A use-package inspired plugin manager for Neovim. Uses native packages, supports Luarocks dependencies, written in Lua, allows for expressive config

MIT License

7.87k stars 267 forks source link

Fix: use vim.notify in packer.log #1044

Closed wbthomason closed 2 years ago

wbthomason commented 2 years ago

vim.notify exists now, and is preferable to the old echom method of logging. Some parts of packer used vim.notify directly, while some used log. This led to inconsistent output.

This PR makes the log API use vim.notify internally, and moves manual uses of vim.notify to use the log API.

Use vim.notify instead of echom to log to console
Move manual uses of vim.notify to use the log API

wbthomason commented 2 years ago

Note that there's still a manual use of vim.notify in the compiled code. I left this because (1) we don't want to have to pull in packer.log in the compiled output and (2) all the compiled stuff will go away soon.