restructure to make it work with antigen #21

Closed
wants to merge 7 commits into
from

3 participants

@bitcloud

because antigen just loads every file in the base folder to initialise plugins, I moved the internal libs to its own folder.

@bitcloud

sorry for the commit mess. I had a small disagreement with my repo :)
now it should work as an oh-my-zsh plugin as well.
originally I wanted to do a separate branch for that, but muscle memory kicked in and pushed the changes to this pull-request as well :)

@tarruda

did #20 fix this?

@bitcloud

I just checked the patch with my oh-my-zsh test env and it didn't load the plugin by default. The problem is, that the repo name has the 'zsh-' prefix. So when you clone it, it has that prefix as well in the folder name. And the folder name has to match the plugin name.

+/Users/js/.zsh/zsh.sh:34> plugin=zsh-autosuggestions
+/Users/js/.zsh/zsh.sh:35> [ -f /plugins/zsh-autosuggestions/zsh-autosuggestions.plugin.zsh ']'
+/Users/js/.zsh/zsh.sh:37> [ -f /Users/js/.zsh/plugins/zsh-autosuggestions/zsh-autosuggestions.plugin.zsh ']'

so to stick with the repo name, renaming the plugin file and add the prefix would fix that.
Antigen is already working.

And when I tried it, I had issues without sourcing the actual file in the plugin file. Thats why I added the "source" as well. But seems to work now. Don't know where those issues came from...

restructuring issn't really necessary due to the plugin file now.

@faceleg faceleg closed this Dec 9, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment