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

Document ft_func with an example #1066

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

Flimm
Copy link

@Flimm Flimm commented Nov 22, 2023

ft_func and load_ft_func need to be configured together. This commit updates the example to include both settings at once.

I know it's a small change, but it took me quite some time to work this out. So I hope the example configuration will be helpful to others and will save them time.

Flimm and others added 2 commits November 22, 2023 15:24
ft_func and load_ft_func need to be configured together. This commit
updates the example to include both settings at once.
@L3MON4D3
Copy link
Owner

Ah, sorry, it's hard to know wherebto omit for brevity and where just that could cost others time😅
Thank you for the change, I appreciate it!

Could you also add a comment that explains that without ft_func there will be no apparent effect on luasnips behaviour? (and squash, so it's one commit+doc-generation)

@L3MON4D3
Copy link
Owner

L3MON4D3 commented Dec 1, 2023

Ping
If you don't have time, I can do the small modifications as well

@Flimm
Copy link
Author

Flimm commented Dec 5, 2023

Ping If you don't have time, I can do the small modifications as well

Yes, please! 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.

2 participants