Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Error running config: 'group' must be a string with default config #169

Closed
tim-hilt opened this issue Mar 9, 2022 · 8 comments
Closed

Comments

@tim-hilt
Copy link

tim-hilt commented Mar 9, 2022

This is the full error:

packer.nvim: Error running config for github-nvim-theme: ...packer/start/github-nvim-theme/lua/github-theme/util.lua:115: 'group' must be a string

And this is my configuration for github-nvim-theme:

  use {
    "projekt0n/github-nvim-theme",
    config = function()
      require("github-theme").setup({})
    end
  }
  -- related:
  use {
    "nvim-lualine/lualine.nvim",
    after = "github-nvim-theme",
    requires = { "kyazdani42/nvim-web-devicons", opt = true },
    config = function()
      require("lualine").setup({ options = { theme = "auto" } })
    end
  }

You can take a look at the config in context here.

@tim-hilt
Copy link
Author

tim-hilt commented Mar 9, 2022

Update: pinning to commit 4ad832e makes the error go away, but now lualine looks funny.

@ful1e5
Copy link
Member

ful1e5 commented Mar 10, 2022

Duplicate of #168

@ful1e5 ful1e5 marked this as a duplicate of #168 Mar 10, 2022
@ful1e5 ful1e5 closed this as completed Mar 10, 2022
@ful1e5 ful1e5 marked this as not a duplicate of #168 Mar 10, 2022
@tim-hilt
Copy link
Author

tim-hilt commented Mar 10, 2022

@ful1e5 I’m using the newest Neovim-commit, so that shouldn’t be a duplicate. If you navigate to misc.lua in my config you can see, that I’m already using native augroups. The error even looks different than the one from #168. Also pinning to the commit mentioned in the linked issue didn’t solve the issue for me.

could you reopen the issue please?

@ful1e5 ful1e5 reopened this Mar 10, 2022
@tim-hilt
Copy link
Author

Thank you!

@ful1e5
Copy link
Member

ful1e5 commented Mar 10, 2022

Should you try with 508d430 ?

@tim-hilt
Copy link
Author

I’ll check it out later today. Reverting seems to be a real pity, however holding out native aus should also benefit users, that are not using nightly Neovim.

@ful1e5
Copy link
Member

ful1e5 commented Mar 10, 2022

I will manage to add native autocmds in short with version condition.

@tim-hilt
Copy link
Author

That's a good approach. I checked the newest commit and it seems to work again for me, so this issue can be closed.

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

No branches or pull requests

2 participants