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

chore: remove redundant comment #1307

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

nhld
Copy link
Contributor

@nhld nhld commented Jan 13, 2025

This is a duplicate comment as it's already mentioned in the mason part.

Copy link

@BALAJI24092001 BALAJI24092001 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For anyone who's starting Neovim with Kickstart might consider this comment useful. I didn't understand when I started, and this helped me.

@nhld
Copy link
Contributor Author

nhld commented Jan 15, 2025

@BALAJI24092001

The recent PR #1298 added a note with the same concept, as the mason config is now using opts = {} - this is now the first occurrence of using opts with an empty table in init.lua. So the fidget's note seems kinda redundant no?

kickstart.nvim/init.lua

Lines 459 to 469 in db4867a

-- Automatically install LSPs and related tools to stdpath for Neovim
-- Mason must be loaded before its dependents so we need to set it up here.
-- NOTE: `opts = {}` is the same as calling `require('mason').setup({})`
{ 'williamboman/mason.nvim', opts = {} },
'williamboman/mason-lspconfig.nvim',
'WhoIsSethDaniel/mason-tool-installer.nvim',
-- Useful status updates for LSP.
-- NOTE: `opts = {}` is the same as calling `require('fidget').setup({})`
{ 'j-hui/fidget.nvim', opts = {} },

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.

2 participants