neovim / node-client

Nvim Node.js client and plugin host
https://neovim.io/node-client/
MIT License
492 stars 53 forks source link

logging: default to log level "error" instead of "debug" #334

Open justinmk opened 8 months ago

justinmk commented 8 months ago

Problem

Currently we default to "debug" log level: https://github.com/neovim/node-client/blob/aac45ab94b36add3aecd238bd7b8c845fde81399/packages/neovim/src/utils/logger.ts#L3

That seems a bit strange, it requires all consumers to undo that for production. Example: https://github.com/vscode-neovim/vscode-neovim/blob/7592f1909467907da934e1748b1b7d5904dba67d/webpack.config.js#L50-L53

Expected behavior

Change default log-level. Update docs and possibly provide a warning during a "transitional" period.