search

zend-patterns / ZendServerPuppet

ZendServerPuppet
Apache License 2.0
6 stars 16 forks source link

Zend Server ensure => latest by default #36

Closed ubellavance closed 7 years ago

ubellavance commented 7 years ago

This module would install the latest version of Zend Server, but also upgrade if possible. I think that it should be ensure => installed instead, to avoid bad surprises.

https://github.com/zend-patterns/ZendServerPuppet/blob/master/manifests/install.pp#L30

ubellavance commented 7 years ago

@mkherlakian Opinion?

ubellavance commented 7 years ago

@mkherlakian unless you state otherwise, I'll change latest to installed in #50 or the next PR.

ubellavance commented 7 years ago

Fixed in #55