From 3c26fe7fec6568f26b9d52cff3c96f28f0d23bb8 Mon Sep 17 00:00:00 2001 From: William Boman Date: Sat, 2 Oct 2021 23:40:28 +0200 Subject: lower default log level to INFO (#119) --- README.md | 2 +- doc/nvim-lsp-installer.txt | 2 +- lua/nvim-lsp-installer/settings.lua | 2 +- lua/nvim-lsp-installer/ui/status-win/init.lua | 3 ++- plugin/nvim-lsp-installer.vim | 4 +--- 5 files changed, 6 insertions(+), 7 deletions(-) diff --git a/README.md b/README.md index 1008ee9f..56fec2d9 100644 --- a/README.md +++ b/README.md @@ -130,7 +130,7 @@ local DEFAULT_SETTINGS = { -- Controls to which degree logs are written to the log file. It's useful to set this to vim.log.levels.DEBUG when -- debugging issues with server installations. - log_level = vim.log.levels.WARN, + log_level = vim.log.levels.INFO, -- Whether to allow LSP servers to share the same installation directory. For some servers, this effectively causes -- more than one server to be installed (and uninstalled) when executing `:LspInstall` and `:LspUninstall`. For diff --git a/doc/nvim-lsp-installer.txt b/doc/nvim-lsp-installer.txt index fceb1c76..29f26075 100644 --- a/doc/nvim-lsp-installer.txt +++ b/doc/nvim-lsp-installer.txt @@ -147,7 +147,7 @@ The following settings are applied by default. > -- Controls to which degree logs are written to the log file. It's useful to set this to vim.log.levels.DEBUG when -- debugging issues with server installations. - log_level = vim.log.levels.WARN, + log_level = vim.log.levels.INFO, -- Whether to allow LSP servers to share the same installation directory. For some servers, this effectively causes -- more than one server to be installed (and uninstalled) when executing `:LspInstall` and `:LspUninstall`. For diff --git a/lua/nvim-lsp-installer/settings.lua b/lua/nvim-lsp-installer/settings.lua index eb01c78c..1d7fa283 100644 --- a/lua/nvim-lsp-installer/settings.lua +++ b/lua/nvim-lsp-installer/settings.lua @@ -12,7 +12,7 @@ local DEFAULT_SETTINGS = { -- Controls to which degree logs are written to the log file. It's useful to set this to vim.log.levels.DEBUG when -- debugging issues with server installations. - log_level = vim.log.levels.WARN, + log_level = vim.log.levels.INFO, -- Whether to allow LSP servers to share the same installation directory. For some servers, this effectively causes -- more than one server to be installed (and uninstalled) when executing `:LspInstall` and `:LspUninstall`. For diff --git a/lua/nvim-lsp-installer/ui/status-win/init.lua b/lua/nvim-lsp-installer/ui/status-win/init.lua index 143dcba8..b82aef0a 100644 --- a/lua/nvim-lsp-installer/ui/status-win/init.lua +++ b/lua/nvim-lsp-installer/ui/status-win/init.lua @@ -283,7 +283,7 @@ local function init(all_servers) state.servers[server.name].installer.is_running = true end) - log.debug("Starting install", server.name, requested_version) + log.fmt_info("Starting install server_name=%s, requested_version=%s", server.name, requested_version or "N/A") server:install_attached({ requested_server_version = requested_version, @@ -302,6 +302,7 @@ local function init(all_servers) end, }, }, function(success) + log.fmt_info("Installation completed server_name=%s, success=%s", server.name, success) mutate_state(function(state) if success then -- release stdout/err output table.. hopefully ¯\_(ツ)_/¯ diff --git a/plugin/nvim-lsp-installer.vim b/plugin/nvim-lsp-installer.vim index b0a77865..2cd3485b 100644 --- a/plugin/nvim-lsp-installer.vim +++ b/plugin/nvim-lsp-installer.vim @@ -41,9 +41,7 @@ function! s:LspInstallInfo() abort endfunction function! s:LspInstallLog() abort - let l:log_file = luaeval("require'nvim-lsp-installer.log'.outfile") - tabnew - exe 'e ' .. l:log_file + exe 'tabnew ' .. luaeval("require'nvim-lsp-installer.log'.outfile") endfunction command! -nargs=+ -complete=custom,s:LspInstallCompletion LspInstall exe s:LspInstall("") -- cgit v1.2.3-70-g09d2