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

snippets/luasnip: add setupOpts #549

Merged
merged 7 commits into from
Jan 12, 2025

Conversation

LilleAila
Copy link
Contributor

@LilleAila LilleAila commented Jan 11, 2025

Added setupOpts to vim.snippets.luasnip.

Right now, there is a soft dependency on nvim-cmp, because setting lazy = true, makes it so that it won't load if nvim-cmp is not present and configured. I will not fix that in this pr, as users can set vim.lazy.plugins.luasnip.event = "BufEnter" themselves.

Also fixed a bug where the plugin previously would not get loaded, as
lazy was set to true without a trigger event.
@LilleAila LilleAila requested a review from NotAShelf as a code owner January 11, 2025 13:03
NotAShelf
NotAShelf previously approved these changes Jan 11, 2025
@NotAShelf NotAShelf requested a review from diniamo January 11, 2025 14:30
modules/plugins/snippets/luasnip/config.nix Outdated Show resolved Hide resolved
docs/release-notes/rl-0.8.md Outdated Show resolved Hide resolved
Copy link
Collaborator

@diniamo diniamo left a comment

Choose a reason for hiding this comment

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

There else seem to be conflicts

modules/plugins/snippets/luasnip/config.nix Outdated Show resolved Hide resolved
@LilleAila LilleAila requested a review from diniamo January 12, 2025 10:41
diniamo
diniamo previously approved these changes Jan 12, 2025
@diniamo
Copy link
Collaborator

diniamo commented Jan 12, 2025

-.-

@diniamo
Copy link
Collaborator

diniamo commented Jan 12, 2025

@NotAShelf preview action fail, seems to be unrelated

@NotAShelf
Copy link
Owner

I'm aware, refer to my comment on Discord for the CI.

Can't review for the next 10 hours or so, feel free to merge if you feel it's ready.

@diniamo
Copy link
Collaborator

diniamo commented Jan 12, 2025

Can't merge without your review apparently
image

@NotAShelf NotAShelf merged commit 520a065 into NotAShelf:main Jan 12, 2025
12 of 13 checks passed
@NotAShelf
Copy link
Owner

Thank you

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.

4 participants