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

Moved from #include_class to #contain_class #67

Merged
merged 1 commit into from Jan 29, 2014
Merged

Moved from #include_class to #contain_class #67

merged 1 commit into from Jan 29, 2014

Conversation

retr0h
Copy link

@retr0h retr0h commented Jan 15, 2014

Removed per the DEPRECATION warning [1]. Since this repo does not include a
Gemfile.lock, and the Gemfile does not have version requirements, bundle
will install the most recent upstream version of the gems.

I personally suggest adding the Gemfile.lock to the repo. This will guarantee
the exact gem deps are installed on each execution of bundle. However, for
now will keep things as is, and simply added a .gitignore.

[1] DEPRECATION: include_class is deprecated. Use contain_class instead. Called from /Users/jodewey/.rvm/gems/ruby-1.9.3-
p484/gems/rspec-puppet-1.0.1/lib/rspec-puppet/matchers/include_class.rb:7:in `block (2 levels) in <module:ManifestMatchers>'.

Removed per the DEPRECATION warning [1].  Since this repo does not include a
Gemfile.lock, and the Gemfile does not have version requirements, `bundle`
will install the most recent upstream version of the gems.

I personally suggest adding the Gemfile.lock to the repo.  This will guarantee
the exact gem deps are installed on each execution of `bundle`.  However, for
now will keep things as is, and simply added a .gitignore.

    [1] DEPRECATION: include_class is deprecated. Use contain_class instead. Called from /Users/jodewey/.rvm/gems/ruby-1.9.3-
    p484/gems/rspec-puppet-1.0.1/lib/rspec-puppet/matchers/include_class.rb:7:in `block (2 levels) in <module:ManifestMatchers>'.
hunner added a commit that referenced this pull request Jan 29, 2014
Moved from `#include_class` to `#contain_class`
@hunner hunner merged commit 5dc2a10 into puppetlabs:master Jan 29, 2014
cegeka-jenkins pushed a commit to cegeka/puppet-haproxy that referenced this pull request Dec 19, 2017
Moved from `#include_class` to `#contain_class`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants