Skip to content

s:projectionist_detect: only handle .vim files #36

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

blueyed
Copy link
Contributor

@blueyed blueyed commented Apr 9, 2018

This avoids sourcing the autoload file for scriptease#locate when not
editing Vim files.

The projectionist setup gets also done for doc/*.txt, which gets
skipped now. This could be changed/enhanced if needed.

@tpope
Copy link
Owner

tpope commented Apr 9, 2018

That autoload is unfortunate, but I do want this for doc files, README files, Python files, everything really.

If this really is a pain point, I'd accept a bit of duplication. Maybe give scriptease#locate a second optional parameter for candidates, then duplicate just the candidate portion in the plugin file.

This avoids sourcing the autoload file for `scriptease#locate` when not
editing relevant files.
@blueyed
Copy link
Contributor Author

blueyed commented Apr 10, 2018

Amended.
Also made it work for me after, by using resolve(): I have symlinks in my &rtp.

@XVilka
Copy link

XVilka commented Aug 7, 2019

@blueyed can you please rebase this one?

@tpope
Copy link
Owner

tpope commented Aug 7, 2019

Remove the resolve() calls you tried to sneak in while you're at it. Make your case for that separately.

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.

3 participants