Wrapper

[Phil-Friderici]

Since v2.0 it is not possible to use data from hiera. This adds init.pp as wrapper class to allow usage of hiera features like merging through levels.

[Phil-Friderici]

Oh, I forgot to update the README. If you like this commit, please let me know and I will add the documentation update.

[petems]

Hi @Phil-Friderici! Looks cool!

I'm not sure I like hiera only abstract classes in modules though, I prefer that code goes into roles and profiles...

However, you could add the code to the examples/ directory, rename it to hiera.pp as an example of how to do it and add a note to the README about it?

[Phil-Friderici]

@petems Sure I'll update README and add an example.

[Phil-Friderici]

I have updated the README accordingly. With the wrapper class we now have an alternative upgrade path for v1.0.1 users that I pointed out too.
Examples for hiera usage and wrapper usage are to be find in the examples directory.

With the wrapper class we would be able to upgrade to the upstream version of your module again :)
As the wrapper class is passive it will now allow to choose which way to go. Hope you are able to not dislike it 😏

Thanks and have a nice weekend!

[Phil-Friderici]

guessing this was intended

[petems]

Hey @Phil-Friderici! Sorry it took so long, this looks great! 👍

[Phil-Friderici]

@petems Thanks a lot for reviewing/asking/merging!
Now we can upgrade back to upstream again :)

[petems]

@Phil-Friderici No problem! Should be able to do a release to the forge tonight, if I dont in a few days remind me! 😄

[Phil-Friderici]

@petems our Puppetfile points to your Github source anyway ;)

[Phil-Friderici]

@petems do you mind to create a new release tag ?
Wouldn't like to use a specific commit number in our Puppetfiles for modules.

Thanks & have phun
Phil

[petems]

Done! 😄

https://github.com/petems/puppet-swap_file/releases/tag/v2.2.0

[Phil-Friderici]

THANK YOU, Thank You, thank you ☺️