make it possible tu use :GpWhisper <LANGUAGE>

[teto]

so this can be changed on the fly.

Ideally I would like to be able to pass it an existing file too but I wanted to do a minimum first contribution. Also I develop on top of #93 which doesn't help

[teto]

@Robitx you mentioned pending comments but I dont see any ? could you precise please

[Robitx]

@teto sorry, I've forgot the review in the pending state 🤦‍♂️

[teto]

ha yeah the log commands interrupt the user, quite bad ^^'''
I wish we could have logging to a file though, it's just too helpful to diagnose issues.
What do you think of vendoring https://github.com/nvim-neorocks/rocks.nvim/blob/master/lua/rocks/log.lua (or something else but there doesn't seem to be a clear winner among logging library yet, plenary.nvim''s is bad) ? could be for another PR or this one.

[Robitx]

@teto another PR, I can do the logging soon but I'm currently rewriting the Prompt function used in the hooks because the signature got a bit unwieldy with multiple providers and leads to bugs. (Sadly another necessary breaking change).

For logging I wouldn't drag another dependency just for this

• adjusting M._log to append all messages to some configurable log file (defaulting to /tmp/) with max history size should be trivial.
• all the messages could also be put into M._log_history list which would be part of the output of :GpInspectPlugin
• and add M.debug which goes only into log file and _log_history

gp.nvim/lua/gp/init.lua

Lines 547 to 574 in 0878f2f

	---@param msg string # message to log
	---@param kind string # hl group to use for logging
	---@param history boolean # whether to add the message to history
	M._log = function(msg, kind, history)
	vim.schedule(function()
	vim.api.nvim_echo({
	{ M._Name .. ": " .. msg, kind },
	}, history, {})
	end)
	end
	-- nicer error messages using nvim_echo
	---@param msg string # error message
	M.error = function(msg)
	M._log(msg, "ErrorMsg", true)
	end
	-- nicer warning messages using nvim_echo
	---@param msg string # warning message
	M.warning = function(msg)
	M._log(msg, "WarningMsg", true)
	end
	-- nicer plain messages using nvim_echo
	---@param msg string # plain message
	M.info = function(msg)
	M._log(msg, "Normal", true)
	end

[teto]

rebased, addressed comments and updated the doc. Tested locally with French and English. Worked nicely.

For logging I wouldn't drag another dependency just for this

I was not proposing to add a dependency (yet :) ) but just to copy/paste that file into our codebase.

[Robitx]

@teto btw. the doc/gp.nvim.txt is made automatically by a CI function, README is the source of truth from which it's made.

[teto]

How do you prefer to work ? once approved, the committer can selfmerge ?

[Robitx]

@teto I've enabled auto merge so next time, the approved PR should get merged automatically. With this one feel free to merge it.