The Puppet coding style requires inter alia:
- Must single quote all resource names and their attribute (unless they contain a variable, of course).
This grossly clashes with the status quo:
[tim@passepartout ~/src/operations/puppet]$ git grep "ensure *=> *\([a-z]\+\)" | wc -l 1378 [tim@passepartout ~/src/operations/puppet]$ git grep "ensure *=> *\('[a-z]\+'\)" | wc -l 465 [tim@passepartout ~/src/operations/puppet]$
Before I submit a patch to change 1378 lines :-) (or rather multiple patches), I'd like to make sure that there is consensus that it should indeed be written:
ensure => 'present',
Also, there are 48 occurences of ensure => installed/ensure => 'installed'. installed is equivalent to present, and I like to replace installed with present (or rather 'present') for consistency as well.