puppet-php icon indicating copy to clipboard operation
puppet-php copied to clipboard

Published 20 hours ago • verified publisher icon thias

Pahoughton test

Open pahoughton opened this issue 11 years ago • 2 comments

Minor fix in ini.pp and cli.pp. Changed so package is installed before ini. The package creates the ini files subdirectory on Debian.

Added travic-ci basic unit testing.

Thanks for providing this module!

pahoughton avatar May 10 '14 07:05 pahoughton

Hmm, some changes make sense, but others don't, such as changing all of the 'include' to 'class' when not passing parameters (I'm quite certain that this syntax makes it fail when then referencing the class to be included multiple times with certain versions of puppet), or having the build status of your branch displayed on mine...

Could you maybe break this PR into various, sequentially, starting with what you consider the most important?

thias avatar May 12 '14 11:05 thias

Major rework pending, lets review the next commit and decide what we want (this weekend).

include vs class - agree this is a issue, looking for the best answer still.

Enjoy Today, TTFN, paul :+} Facebook LinkedIn

On May 12, 2014, at 05:09 , Matthias Saou [email protected] wrote:

Hmm, some changes make sense, but others don't, such as changing all of the 'include' to 'class' when not passing parameters (I'm quite certain that this syntax makes it fail when then referencing the class to be included multiple times with certain versions of puppet), or having the build status of your branch displayed on mine...

Could you maybe break this PR into various, sequentially, starting with what you consider the most important?

— Reply to this email directly or view it on GitHub.

pahoughton avatar May 15 '14 19:05 pahoughton