CopilotC-Nvim / CopilotChat.nvim

Chat with GitHub Copilot in Neovim
https://copilotc-nvim.github.io/CopilotChat.nvim/
GNU General Public License v3.0
1.44k stars 67 forks source link

README improvement: `opts = {}` key must be specified for lazy.nvim installation #341

Closed oryband closed 3 months ago

oryband commented 3 months ago

The following configuration doesn't work without at least adding a minimal, empty opts = {} section. Please add to readme.

  {
    "CopilotC-Nvim/CopilotChat.nvim",
    branch = "canary",
    dependencies = {
      { "zbirenbaum/copilot.lua" },
      { "nvim-lua/plenary.nvim" },
    },
    -- opts = {},  -- doesn't load or start plugin without this uncommented
  },
deathbeam commented 3 months ago

Interesting, so it doesnt load or does it crash on something? Not super familiar with lazy.nvim

oryband commented 3 months ago

I don't see any errors, the :CopilotChat commands just aren't available. How can I check this? Note I'm using AstroNvim v4 https://astronvim.com/

deathbeam commented 3 months ago

I don't see any errors, the :CopilotChat commands just aren't available. How can I check this? Note I'm using AstroNvim v4 https://astronvim.com/

If you dont see any errors then its probs not some weird crash related to that. But otherwise not rly familiar with astronvim either, but I would assume the errors would just be in :messages if any.

folke commented 3 months ago

If you don't add opts = {}, then CopilotChat's setup(opts) would not have run.

folke commented 3 months ago

The readme does mention the opts = {} no?

deathbeam commented 3 months ago

Alright so its intended behaviour then, was mostly worried that its on the plugin side in some way. And yea we even mention the opts, completely forgot :d

oryband commented 3 months ago

I understand the reason, it was just unclear to me that it has to be added. I wanted to use the default opts so i just removed this key entirely - which practically disabled the plugin and was unintuitive. I spent 30m figuring this out!