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

Fix Digester to require its dependencies #2594

Merged
merged 1 commit into from
Dec 10, 2014
Merged

Conversation

jaymzh
Copy link
Collaborator

@jaymzh jaymzh commented Dec 9, 2014

Without thus using rspec gets you:

uninitialized constant Chef::Digester::Singleton

Without thus using rspec gets you:

```
uninitialized constant Chef::Digester::Singleton
```
@jaymzh
Copy link
Collaborator Author

jaymzh commented Dec 9, 2014

CC @opscode/client-core

@danielsdeleo
Copy link
Contributor

👍

@lamont-granquist
Copy link
Contributor

yeah that one is obvious 👍

@tyler-ball
Copy link
Contributor

👍

@jaymzh
Copy link
Collaborator Author

jaymzh commented Dec 10, 2014

And we have a majority!

jaymzh added a commit that referenced this pull request Dec 10, 2014
Fix Digester to require its dependencies
@jaymzh jaymzh merged commit 1c2579c into chef:master Dec 10, 2014
@jaymzh jaymzh deleted the digester branch December 10, 2014 00:07
jaymzh added a commit that referenced this pull request Dec 16, 2014
Fix Digester to require its dependencies
jaymzh added a commit that referenced this pull request Dec 16, 2014
Merge pull request #2594 from jaymzh/digester
jaym added a commit that referenced this pull request Feb 19, 2015
* origin/12-stable: (83 commits)
  Disable win32 memory leak tests
  Stub windows? check in the unit test to make sure specs are green on windows.
  Fix build version for appveyor to be specific to the 12-stable branch
  More info more version
  Adding appveyor.yml
  Merge pull request #2749 from opscode/jdm/metadata-error-info
  Clearing out DOC_CHANGES from 12-stable because all docs have been updated since 12.0.3 release
  Contribution info for #2658.
  Bump version for 12.0.3 release.
  Merge pull request #2594 from jaymzh/digester
  Bump version number for 12.0.2 release.
  add CHANGELOG.md
  fix apt default_release attribute broken in 12.0
  Merge pull request #2650 from opscode/sersut/contrib-json
  Merge pull request #2642 from opscode/btm/site_install_json
  Changelog for #2621
  Merge pull request #2645 from opscode/sersut/contrib-2634
  Merge pull request #2634 from BackSlasher/repair-subversion-command
  Fixing homebrew package to use provider resolver per code review
  Update CHANGELOG for #2630
  ...

Conflicts:
	.travis.yml
	CHANGELOG.md
	DOC_CHANGES.md
	appveyor.yml
	lib/chef/application.rb
	lib/chef/dsl/recipe.rb
	lib/chef/knife/bootstrap.rb
	lib/chef/knife/core/bootstrap_context.rb
	lib/chef/node/attribute.rb
	lib/chef/node/attribute_collections.rb
	lib/chef/node/immutable_collections.rb
	lib/chef/resource.rb
	lib/chef/run_context.rb
	lib/chef/version.rb
	spec/functional/dsl/reboot_pending_spec.rb
	spec/functional/event_loggers/windows_eventlog_spec.rb
	spec/functional/resource/link_spec.rb
	spec/support/platform_helpers.rb
	spec/unit/knife_spec.rb
	spec/unit/mixin/deep_merge_spec.rb
	spec/unit/mixin/shell_out_spec.rb
	spec/unit/node/attribute_spec.rb
	spec/unit/node_spec.rb
	spec/unit/provider/package/apt_spec.rb
	spec/unit/provider/service/systemd_service_spec.rb
	spec/unit/provider_resolver_spec.rb
	spec/unit/recipe_spec.rb
	spec/unit/resource/resource_notification_spec.rb
	spec/unit/run_context_spec.rb
@chef chef locked and limited conversation to collaborators Nov 16, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants