Skip to content
This repository was archived by the owner on Jan 15, 2024. It is now read-only.

Puppet 5 support#179

Merged
Ma27 merged 3 commits intomasterfrom
puppet-5
May 1, 2018
Merged

Puppet 5 support#179
Ma27 merged 3 commits intomasterfrom
puppet-5

Conversation

@Ma27
Copy link
Copy Markdown
Collaborator

@Ma27 Ma27 commented Apr 14, 2018

Overview

Updates our package set, uses Puppet5 for development by default.

For further details have a look at the commit messages...

Ma27 added 2 commits April 14, 2018 16:22
No automatic merging for facts defined in global test scope (https://github.com/rodjek/rspec-puppet/blob/master/CHANGELOG.md#263)

Full evaluation for resource definitions, so now
`::nodejs::instance::pkgs` needs to be evaluated before (or at least the
needed packages with Puppet::Package)
@Ma27 Ma27 requested a review from willdurand April 14, 2018 15:32
@Ma27 Ma27 added the in review label Apr 14, 2018
Puppet 5.x ships with Ruby 2.4, but older Puppet versions (we still
support Puppet 3.x) are obvioulsy incompatible.

See affected build at https://travis-ci.org/willdurand/puppet-nodejs/builds/366517540
@Ma27
Copy link
Copy Markdown
Collaborator Author

Ma27 commented Apr 14, 2018

@willdurand the tests for Puppet3 are currently broken (I'm working on a fix, seems to be a Travis issue), would be cool if you could have a short look at this :)

@willdurand
Copy link
Copy Markdown
Owner

@willdurand the tests for Puppet3 are currently broken (I'm working on a fix, seems to be a Travis issue), would be cool if you could have a short look at this :)

ah sorry I missed that, will review it tomorrow.

@Ma27 Ma27 merged commit 2ae1bd0 into master May 1, 2018
@Ma27 Ma27 deleted the puppet-5 branch May 1, 2018 15:31
@Ma27 Ma27 removed the in review label May 1, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants