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

(maint) Update Gemfile and README for Ruby 2.5/2.4 #293

Merged
merged 2 commits into from
Nov 18, 2019

Conversation

glennsarti
Copy link
Contributor

Previously it was not possible to bundle install on Ruby 2.5 with Puppet 5 on
Windows due to FFI restrictions. This commit updates the Gemfile to use the
appropriate default version of Puppet, based on the Ruby version

ActiveSupport (Which is used by github log generator) v6.x requires Ruby 2.5.
This commit adds a per-ruby version requirement for activesupport which will
allow it to be bundled correctly for Ruby 2.5 and 2.4.


Many of the testing frameworks are not supported on Windows. This commit
updates the README to explain that even though the project can be bundle
installed correctly, the tests cannot be run, until such time as the tests
are modified:


Note this PR also removes some trailing whitespace in the readme.

Previously it was not possible to bundle install on Ruby 2.5 with Puppet 5 on
Windows due to FFI restrictions.  This commit updates the Gemfile to use the
appropriate default version of Puppet, based on the Ruby version

ActiveSupport (Which is used by github log generator) v6.x requires Ruby 2.5.
This commit adds a per-ruby version requirement for activesupport which will
allow it to be bundled correctly for Ruby 2.5 and 2.4.
Many of the testing frameworks are not supported on Windows.  This commit
updates the README to explain that even though the project can be bundle
installed correctly, the tests cannot be run, until such time as the tests
are modified:

- Expect is a bash only utility
- Aruba is not supported on Windows cucumber/aruba#661
@bastelfreak
Copy link
Member

thanks for the PR @glennsarti !

@bastelfreak bastelfreak merged commit 89f8820 into voxpupuli:master Nov 18, 2019
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.

2 participants