Skip to content
This repository was archived by the owner on Dec 18, 2025. It is now read-only.

Add setup instructions for integrating cspell with lazyvim#76

Merged
davidmh merged 1 commit intodavidmh:mainfrom
daehyunpy:main
Feb 2, 2025
Merged

Add setup instructions for integrating cspell with lazyvim#76
davidmh merged 1 commit intodavidmh:mainfrom
daehyunpy:main

Conversation

@daehyunpy
Copy link
Copy Markdown
Contributor

I love your plugin! However, as a Neovim newbie, it took me some time to set it up. LazyVim is quite popular among beginners, and including setup instructions could make the plugin even more accessible.

I hope you’ll review my PR and consider merging it.

Copy link
Copy Markdown
Owner

@davidmh davidmh left a comment

Choose a reason for hiding this comment

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

Hi @daehyunpy, looks good overall.

Would you mind addressing my comment above and amending the commit message to comply with the commitlint rules?

header must not be longer than 100 characters, current length is 126 [header-max-length]

Comment thread README.md Outdated
Comment on lines +13 to +16
{
"davidmh/cspell.nvim",
event = "VeryLazy",
},
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

We don't really need to list the plugin as it's own entry, being listed in the none-ls dependencies is enough to have lazyvim install it.

@daehyunpy
Copy link
Copy Markdown
Contributor Author

I have mended your suggestions!

@daehyunpy daehyunpy requested a review from davidmh February 2, 2025 06:58
Copy link
Copy Markdown
Owner

@davidmh davidmh left a comment

Choose a reason for hiding this comment

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

Thanks!

@davidmh davidmh merged commit fb104ec into davidmh:main Feb 2, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants