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

Adds support to change the top level directory for plugins #15

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

Conversation

logicminds
Copy link
Contributor

  • previously by default, pluginator would only look in the directory
    named plugins. While this is a great notion, many existing plugin
    systems do not follow this path.
    This attempts to allow the user to change the default plugins
    directory to something different.

    Example: lib/plugins/my_group/plugin_name.rb can now be
    lib/alt_plugins/my_group/plugin_name.rb.

@logicminds
Copy link
Contributor Author

I can't test locally due to #14 so not sure if the tests pass. Just wanted to throw this up for review.

@logicminds logicminds force-pushed the plugins_dir branch 6 times, most recently from ed71946 to cd8cd13 Compare June 14, 2017 02:40
  * Fixes rvm#13 - use different plugin path
  * previously by default, pluginator would only look in the directory
    named plugins.  While this is a great notion, many existing plugin
    systems do not follow this path.
    This attempts to allow the user to change the default plugins
    directory to something different.

    Example: lib/plugins/my_group/plugin_name.rb  can now be
             lib/alt_plugins/my_group/plugin_name.rb.
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 9209259 on nwops:plugins_dir into e205d0e on rvm:master.

@rvm rvm deleted a comment from coveralls Jun 14, 2017
@rvm rvm deleted a comment from coveralls Jun 14, 2017
@rvm rvm deleted a comment from coveralls Jun 14, 2017
@rvm rvm deleted a comment from coveralls Jun 14, 2017
@rvm rvm deleted a comment from coveralls Jun 14, 2017
@rvm rvm deleted a comment from coveralls Jun 14, 2017
@rvm rvm deleted a comment from coveralls Jun 14, 2017
@rvm rvm deleted a comment from coveralls Jun 14, 2017
@mpapis
Copy link
Member

mpapis commented Jun 14, 2017

great idea, I will try to provide feedback in next 9-10 hours

@mpapis mpapis mentioned this pull request Jun 14, 2017
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