Skip to content

s:projectionist_detect: only handle .vim files#36

Open
blueyed wants to merge 1 commit into
tpope:masterfrom
blueyed:detect
Open

s:projectionist_detect: only handle .vim files#36
blueyed wants to merge 1 commit into
tpope:masterfrom
blueyed:detect

Conversation

@blueyed

@blueyed blueyed commented Apr 9, 2018

Copy link
Copy Markdown
Contributor

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

tpope commented Apr 9, 2018

Copy link
Copy Markdown
Owner

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

blueyed commented Apr 10, 2018

Copy link
Copy Markdown
Contributor Author

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

@XVilka

XVilka commented Aug 7, 2019

Copy link
Copy Markdown

@blueyed can you please rebase this one?

@tpope

tpope commented Aug 7, 2019

Copy link
Copy Markdown
Owner

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