-
-
Notifications
You must be signed in to change notification settings - Fork 374
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
quick fix for failing travis builds #373
Conversation
.travis.yml
Outdated
@@ -38,7 +38,8 @@ env: | |||
- PUPPET_GEM_VERSION="~> 4.6.0" | |||
- PUPPET_GEM_VERSION="~> 4.7.0" | |||
- PUPPET_GEM_VERSION="~> 4.8.0" | |||
- PUPPET_GEM_VERSION="~> 4" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we want to keep this so that we always test against the newest version
.travis.yml
Outdated
@@ -38,7 +38,8 @@ env: | |||
- PUPPET_GEM_VERSION="~> 4.6.0" | |||
- PUPPET_GEM_VERSION="~> 4.7.0" | |||
- PUPPET_GEM_VERSION="~> 4.8.0" | |||
- PUPPET_GEM_VERSION="~> 4" | |||
- PUPPET_GEM_VERSION="~> 4.9.0" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for adding 4.9 and 4.10
Gemfile
Outdated
@@ -18,7 +18,7 @@ else | |||
gem 'facter', :require => false | |||
end | |||
|
|||
gem 'puppetlabs_spec_helper', '>= 1.2.0', :require => false | |||
gem 'puppetlabs_spec_helper', '>= 1.2.0', '<= 1.2.2', :require => false |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
gem 'puppetlabs_spec_helper', '2.0.2', :require => false if RUBY_VERSION >= '1.8.7' && RUBY_VERSION < '1.9'
gem 'puppetlabs_spec_helper', '>= 2.0.0', :require => false if RUBY_VERSION >= '1.9'
anchor { 'python::end': } | ||
anchor { 'python::begin': } | ||
-> class { 'python::install': } | ||
-> class { 'python::config': } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good to me
Thanks for the help; it should pass for real this time. 👍 |
@7yl4r seems like |
You're exactly right, @flokli . It should be an easy fix I plan to come back around to this project and resolve soon. |
@7yl4r If you currently don't have the time, I could take a look at this. Just let me know :-) |
Thanks everyone! Similar functionality added in PR #376 |
This does two things:
Although the better fix re (1) might be to switch over to using the newer spec_helper, at least this will allow tests to pass in the meantime.
Closes #372 .