Skip to content

fix plugin instantiation #113

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

Merged
merged 1 commit into from
Aug 8, 2016
Merged

fix plugin instantiation #113

merged 1 commit into from
Aug 8, 2016

Conversation

dbu
Copy link
Collaborator

@dbu dbu commented Aug 7, 2016

Q A
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Related tickets #112 (found bug while adding tests there - not adding test here to reduce conflicts)
Documentation -
License MIT

What's in this PR?

Fix how we define the plugin services that have a $config parameter so they work.

Why?

They where broken.

Checklist

  • Updated CHANGELOG.md to describe bugfix

@Nyholm
Copy link
Member

Nyholm commented Aug 8, 2016

Thank you. 👍

Im not sure how you will do with this PR regarding merge conflicts. If there is no issue with your other PR Im happy to merge this.

@dbu dbu merged commit 4408c9a into master Aug 8, 2016
@dbu dbu deleted the bugfix/plugin-config branch August 8, 2016 08:09
@sagikazarmark sagikazarmark modified the milestone: v1.3.0 Aug 16, 2016
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