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

refactor(luasnip): add extended configuration #3525

Draft
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

kylo252
Copy link
Collaborator

@kylo252 kylo252 commented Nov 25, 2022

Description

  • feat(luasnip): add virtual text to signal current insert point

  • feat: added lunarvim snippets and move luasnipt to seaprate config

  • chore: try to satisfy luacheck

  • chore: little fix to the extend

  • refactor(luasnip): simplify structure

How Has This Been Tested?

TBD

@danielo515

This comment was marked as off-topic.

@LostNeophyte

This comment was marked as off-topic.

@danielo515

This comment was marked as off-topic.

@LostNeophyte

This comment was marked as off-topic.

Copy link
Contributor

@opalmay opalmay left a comment

Choose a reason for hiding this comment

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

@kylo252 PR looks good :)
Any specific reason this is still a draft?

lua/lvim/core/luasnip.lua Outdated Show resolved Hide resolved
lua/lvim/core/luasnip.lua Show resolved Hide resolved
lua/lvim/core/luasnip.lua Outdated Show resolved Hide resolved
@opalmay opalmay marked this pull request as ready for review February 6, 2023 19:51
@kylo252 kylo252 marked this pull request as draft February 7, 2023 10:21
@kylo252
Copy link
Collaborator Author

kylo252 commented Feb 7, 2023

@kylo252 PR looks good :) Any specific reason this is still a draft?

waiting on the status of #3670 (comment)

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.

None yet

4 participants