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

Fixing the plugins loader #980

Merged
merged 1 commit into from
Feb 11, 2016
Merged

Fixing the plugins loader #980

merged 1 commit into from
Feb 11, 2016

Conversation

subnetmarco
Copy link
Member

Closes #978.

Also loads the consumer-specific configuration only for those plugins that have no_consumer = false in the schema.

@subnetmarco subnetmarco added pr/status/needs review pr/changes requested Changes were requested to this PR by a maintainer. Please address them and ping back once done. labels Feb 10, 2016
@subnetmarco subnetmarco self-assigned this Feb 10, 2016
@subnetmarco subnetmarco force-pushed the fix/plugins-loader branch 6 times, most recently from 130f1c8 to dd9415f Compare February 11, 2016 00:59
@subnetmarco subnetmarco removed pr/changes requested Changes were requested to this PR by a maintainer. Please address them and ping back once done. pr/status/needs review labels Feb 11, 2016
@subnetmarco subnetmarco added the pr/ready (but hold merge) No more concerns, but do not merge yet (probably a conflict of interest with another PR or release) label Feb 11, 2016
subnetmarco added a commit that referenced this pull request Feb 11, 2016
@subnetmarco subnetmarco merged commit 4374ca1 into next Feb 11, 2016
@subnetmarco subnetmarco deleted the fix/plugins-loader branch February 11, 2016 01:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr/ready (but hold merge) No more concerns, but do not merge yet (probably a conflict of interest with another PR or release)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants