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

Add plugin for linting

Open fredrikaverpil opened this issue 2 years ago • 5 comments

Why this change?

  • There is no linter included in kickstart.
  • Fixes #698

What was changed?

  • Add https://github.com/mfussenegger/nvim-lint as the choice of linter. This is a popular linter that is the default linter in nvim distros such as LazyVim.

fredrikaverpil avatar Mar 05 '24 06:03 fredrikaverpil

I wonder why they decided not to do things with none-ls

brxxlstxrs avatar Mar 06 '24 20:03 brxxlstxrs

What do you think about including this as an optional "kickstart.plugins.lint"? I don't think everyone needs an additional linter plugin, lots of languages provide all the linting you need through the LSP (so I think it just adds noise and confusion for people who are starting off and don't realize the LSP usually gives enough info for a lot of languages).

tjdevries avatar Mar 15 '24 14:03 tjdevries

@tjdevries sure, could do that. 👍

fredrikaverpil avatar Mar 15 '24 16:03 fredrikaverpil

@tjdevries have another look now! 😄

fredrikaverpil avatar Mar 15 '24 16:03 fredrikaverpil

Great PR! Linter as an optional is really the way to go! :)

reandov avatar Mar 16 '24 12:03 reandov

Thanks for your contribution. This is what Teej suggested so I'm gonna merge it.

feoh avatar Mar 18 '24 14:03 feoh