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

Dependency injection #1680

Merged
merged 5 commits into from May 4, 2015
Merged

Dependency injection #1680

merged 5 commits into from May 4, 2015

Conversation

@ralsina
Copy link
Member

@ralsina ralsina commented May 3, 2015

Allow plugins to alter task dependencies.

This allows plugins to:

  1. Set themselves as dependencies for other plugins
  2. Set other plugins as dependencies for themselves
  3. Set other plugins as dependencies for other plugins (IOW: eeeeeevil)
ralsina added 4 commits May 3, 2015
@ralsina ralsina mentioned this pull request May 3, 2015
ralsina added a commit that referenced this pull request May 4, 2015
Dependency injection
@ralsina ralsina merged commit e427fc0 into master May 4, 2015
0 of 3 checks passed
0 of 3 checks passed
continuous-integration/appveyor Waiting for AppVeyor build to complete
Details
continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
continuous-integration/travis-ci/push The Travis CI build is in progress
Details
@ralsina ralsina deleted the dependency-injection branch May 4, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

1 participant