search

Chassis / puppet-wp

This module enables the easy use of WP CLI to control your WordPress site through Puppet manifests
https://chassis.github.io/puppet-wp/
GNU Lesser General Public License v3.0
44 stars 39 forks source link

user parameter was missing from the exec within wp::plugin. #32

Closed benpriestman closed 8 years ago

benpriestman commented 8 years ago

Plugin installation was failing on puppet apply runs due to the warning issued to root users executing wp commands. Set to the same user as is used in wp::command.

rmccue commented 8 years ago

Good catch, thanks for this! Your patch has spaces instead of tabs, can you fix this up please? :)

benpriestman commented 8 years ago

Hmm. The puppet style guide https://docs.puppet.com/guides/style_guide.html#spacing-indentation-and-whitespace and I favour two-spaces rather than tabs. I've changed this for consistency, though.

On 15/04/2016 08:05, Ryan McCue wrote:

Good catch, thanks for this! Your patch has spaces instead of tabs, can you fix this up please? :)

— You are receiving this because you authored the thread. Reply to this email directly or view it on GitHub https://github.com/rmccue/puppet-wp/pull/32#issuecomment-210318461

rmccue commented 8 years ago

Cheers!