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

Major refactor of #chef_role and #chef_search and ChefZero tests #28

Closed
wants to merge 5 commits into from
Closed

Conversation

jwreagor
Copy link

This PR includes major refactoring and testing of #chef_role and #chef_search. It also resolves issue #25.

Specifically (from commit message)...

  • Upgrade chef and capistrano gems
  • Refactor #chef_role so that normal and block based querying work
  • Better support for customization via dependency injecting Search class
  • chef-zero backed test units and environment
  • Proper Gemfile and Rakefile
  • Rake::TestTask verbosity is optional

Now that I see PR #27 it would also fit perfectly with this setup.

I have more things I need and can contribute but I'd like to get this into the gem so I can use it. :)

jwreagor added 3 commits June 25, 2014 12:42
* Upgrade chef and capistrano gems
* Refactor #chef_role so that normal and block based querying work
* Better support for customization via dependency injecting Search class
* chef-zero backed test units and environment
* Proper Gemfile and Rakefile
* Rake::TestTask verbosity is optional
@jwreagor
Copy link
Author

Oh noez! We don't want Travis failing now do we. :) I'll have to look into what I can change on my end to get it working.

@jwreagor
Copy link
Author

:octocat:

@jwreagor
Copy link
Author

Actually the #role call I'm making doesn't work the same in Capistrano 3 and I've got a fix coming.

@jwreagor jwreagor closed this Jul 26, 2014
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.

1 participant