kickstart.nvim icon indicating copy to clipboard operation
kickstart.nvim copied to clipboard

fix: Load guess-indent.nvim

Open rivenirvana opened this issue 8 months ago • 11 comments

rivenirvana avatar May 12 '25 16:05 rivenirvana

Hi @feoh and @oriori1703 ,

This PR is very simple to fix the issue I've caused. It's been more than two weeks since the PR came in. Could you merge this or is there any process to merge?

I was just hoping my mess would be cleared away. 😊

guru245 avatar Jun 01 '25 23:06 guru245

Additional note to maintainers, this makes it so that the documentation right above it does not seem to be immediately related or applicable, as it's now a table and not just a link anymore. Let me know if it needs to be adjusted as well.

rivenirvana avatar Jun 02 '25 02:06 rivenirvana

Hi @feoh and @oriori1703 ,

This PR is very simple to fix the issue I've caused. It's been more than two weeks since the PR came in. Could you merge this or is there any process to merge?

I was just hoping my mess would be cleared away. 😊

I wish I could merge this, but I'm not a maintainer 😅

oriori1703 avatar Jun 03 '25 11:06 oriori1703

This change does not fix the issue for me - in C++ files I have indents of size 2spaces. Adding a line before or after an indented line adds 2tabs. adding

opt = {},
config = function()
  require('guess-indent').setup{}
end,

helps.

CapSel avatar Jun 23 '25 10:06 CapSel

Hello @CapSel

For that issue, I think you should report it to https://github.com/NMAC427/guess-indent.nvim as the issue has nothing to do with this PR.

guru245 avatar Jun 23 '25 10:06 guru245

This change does not fix the issue for me - in C++ files I have indents of size 2spaces. Adding a line before or after an indented line adds 2tabs. adding

opt = {},
config = function()
  require('guess-indent').setup{}
end,

helps.

@CapSel maybe because you have opt instead of opts ?

Crazy-Hopper avatar Aug 18 '25 16:08 Crazy-Hopper

@CapSel maybe because you have opt instead of opts ?

@Crazy-Hopper yes, thank you for pointing that out, that fixed my problem :D

CapSel avatar Aug 18 '25 20:08 CapSel

Lol I struggled to get it working and eventually found what was wrong myself. Also about to create a new issue but then found this, it would indeed be good to merge this pls. Or maybe another idea: you could hint to new users there is something wrong with the config somewhere and they have to find it. That way nothing in the config has to change but users are at least aware there is some small error.

Lucas256-cmd avatar Sep 14 '25 14:09 Lucas256-cmd

Why is this not being merged?

nikagar4 avatar Nov 02 '25 11:11 nikagar4

Why is this not being merged?

the repo is dead, the contributors are burned out and we cant get the maintainer to notice us to take it over.

https://github.com/nvim-lua/kickstart.nvim/pull/1627#issuecomment-3283511227 https://github.com/nvim-lua/kickstart.nvim/pull/1627#issuecomment-3123542324

szechp avatar Nov 03 '25 12:11 szechp

I can confirm this works. Nice pr, fixes a bug.

ChillerDragon avatar Nov 06 '25 09:11 ChillerDragon