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

Move backends and OS helpers to separate directories #36

Merged
merged 5 commits into from Apr 6, 2013

Conversation

Projects
None yet
2 participants
@raphink
Copy link
Contributor

raphink commented Apr 5, 2013

This opens the way to add backends and helpers more easily:

  • The backends were moved to a lib/backend directory and imported by lib/backend.rb;
  • The helpers were moved to a lib/helper directory and imported by lib/helper.rb;
  • The backends are now classes, instantiated with commands. This allows to remove all kind of backend intelligence from matchers.

The goal of this big PR is to move toward a really generic backend system, that might not even be based on commands.

raphink added some commits Apr 5, 2013

Use classes for backends
  Use an helper for backends again
  Instantiate backends with commands
  Remove specific implementations from matchers

@raphink raphink referenced this pull request Apr 5, 2013

Merged

Add a Puppet backend #38

@mizzy mizzy merged commit 739ff34 into mizzy:master Apr 6, 2013

1 check was pending

default The Travis build is in progress
Details

mizzy added a commit that referenced this pull request Apr 6, 2013

mizzy added a commit that referenced this pull request Apr 6, 2013

Revert "Revert pull req #36, #37, #38"
This reverts commit 5df4e16.

This was referenced Apr 6, 2013

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