Add Puppet8x support #32
Conversation
| owner => $edit_uid, | ||
| group => 0, | ||
| mode => '0600', | ||
| require => Package['trocla']; |
There was a problem hiding this comment.
the idea to use the collector is that the collector can also be empty since there might be cases where you do not have the package installed.
| } | ||
| } | ||
| Package<| title == 'trocla' |> -> file { |
| r = has_options ? store.send(trocla_func, key, format, options) : store.send(trocla_func, key, format) | ||
| store.close | ||
| r | ||
| # Functions called from puppet manifests that look like this: |
There was a problem hiding this comment.
any reason for the change in indentation?
|
Thank you, much appreciated! The collector statement is imho still necessary and it is still supported: https://www.puppet.com/docs/puppet/8/lang_collectors.html And other than that it would be greate to at least split the indentation changes into their own commit, since this makes it not so clear what are required changes for puppet 8 and what is (debatable) cosmetic changes. Any chance to get that at least splitted? |
|
The changes for Puppet8 mainly concern variable typing and legacy facts. |
No description provided.