Skip to content
This repository has been archived by the owner on Apr 6, 2021. It is now read-only.

Plugin config #25

Merged
merged 1 commit into from
Sep 28, 2015
Merged

Plugin config #25

merged 1 commit into from
Sep 28, 2015

Conversation

janstenpickle
Copy link
Member

Fixes #11

def global_config
global_option('-c', '--config [CONFIG]', "Add config via command line in the format #{CONFIG_FORMAT}") do |config|
match = config.match(/(?<plugin>\S+):(?<key>\S+)\s+(?<value>\S+)/)
if match
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should combine this and the line above.

@@ -33,6 +45,10 @@ def self.with_graph(&block)
(@with_graph_callbacks ||= []) << block
end

def self.name
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Needs a spec.

@janstenpickle janstenpickle force-pushed the 11-plugin-options branch 5 times, most recently from fa4b001 to 730d745 Compare September 28, 2015 13:30
- add methods to resolve plugin name in the Plugin class
- add the name of the plugin as the default plugin tag
- add a command line task to display the enabled plugin names
- add a global command line option to imput global configuration
- refactor env so configuration is in a separate class
- make the config class return OpenStructs instead of a hash
nathankleyn added a commit that referenced this pull request Sep 28, 2015
@nathankleyn nathankleyn merged commit f2cd66c into master Sep 28, 2015
@nathankleyn nathankleyn deleted the 11-plugin-options branch September 28, 2015 13:36
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants