Skip to content

Conversation

@olimorris
Copy link
Owner

Description

Gemini is marvellous to talk to and a pain in the backside to program with. Instead of returning the placement in lower case like every LLM, it likes to capitalise it...

@olimorris olimorris merged commit 04019d5 into main Feb 18, 2025
2 checks passed
@towry
Copy link

towry commented Feb 22, 2025

@olimorris option placement = "replace" not working anymore after this commit.

Error executing vim.schedule lua callback: ...panion.nvim/lua/codecompanion/strategies/inline/init.lua:442: bad argument #1 to 'lower' (string ex
pected, got nil)
stack traceback:
        [C]: in function 'lower'
        ...panion.nvim/lua/codecompanion/strategies/inline/init.lua:442: in function 'cb'
        .../nvim/lazy/codecompanion.nvim/lua/codecompanion/http.lua:117: in function <.../nvim/lazy/codecompanion.nvim/lua/codecompanion/http.lua
:114>

@olimorris
Copy link
Owner Author

@towry it's working for me:

2025-02-22 16_46_47 - WezTerm

Perhaps your LLM is doing something funky.

@towry
Copy link

towry commented Feb 22, 2025

@olimorris
Copy link
Owner Author

The PR isn't the place to raise an issue. Raise a bug report or better still, investigate the issue and raise an appropriate PR.

@olimorris olimorris deleted the fix/inline-placement-chat branch February 27, 2025 14:32
cleong14 pushed a commit to cleong14/codecompanion.nvim that referenced this pull request May 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants