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

Disable auto detecting changes to snippet files on expansion #967

Merged
merged 7 commits into from Apr 10, 2018

Conversation

SirVer
Copy link
Owner

@SirVer SirVer commented Apr 10, 2018

Instead, trigger an auto command whenever a .snippets file is saved that will reload the snippets. This is a performance improvement with a loss of functionality: externally (i.e. outside of the current Vim instance) generated snippet files will not be picked up until UltiSnips#RefreshSnippets is called.

I decided to not expose a command for this, since I assume it is rarely useful. I think the command would add confusion of when it was supposed to be ran.

Fixes #932.

Instead, trigger an auto command whenever a .snippets file is saved that
will reload the snippets. This is a performance improvement with a loss
of functionality: externally (i.e. outside of the current Vim instance)
generated snippet files will not be picked up until
UltiSnips#RefreshSnippets is called.

I decided to not expose a command for this, since I assume it is rarely
useful. I think the command would add confusion of when it was supposed
to be ran.
@SirVer SirVer changed the title Issue 932 Disable auto detecting changes to snippet files on expansion Apr 10, 2018
@SirVer
Copy link
Owner Author

SirVer commented Apr 10, 2018

@UltiBot merge

@UltiBot
Copy link
Collaborator

UltiBot commented Apr 10, 2018

Merge requested by authorized user SirVer. Merge queue now has a length of 3.

@UltiBot UltiBot merged commit 4eac979 into master Apr 10, 2018
@SirVer SirVer deleted the issue_932 branch April 11, 2018 06:51
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

2 participants