If we've only got the one module here, and no in-line modifications, `#tap` is just going to incur a performance hit (blocks vs. inlining, etc) without any readability benefits _in this particular case_, IMO.

Would rather see:

``` ruby
@modules << PuppetModule.new module_name, version, options
```