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

The watch_config script should directly query plugins to make configuration easier. #121

Open
seamustuohy opened this issue Feb 4, 2016 · 1 comment

Comments

@seamustuohy
Copy link
Contributor

The watch_config script currently has the config files to watch, and their corresponding hard coded.It should directly query plugins to make adding a file to watch for a new plugin easier.

Something like the following where get_config_pairs is a function that creates a namedTuple from a config file or set of environment variables.

plugins = get_config_pairs(os.environ['COPILOT_PLUGIN_DIR'])
for plugin in plugins:
    config_handle.add_config(plugin.conf_file, plugin.service_name)
@seamustuohy seamustuohy changed the title The watch_config script should pull from environment variables or a config file to make adding plugins easier The watch_config script should directly query plugins to make configuration easier. Feb 9, 2016
@seamustuohy
Copy link
Contributor Author

This should query the plugin directory environment variable defined in issue #124 .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant