On 2012-06-29 23:27, jcbollinger wrote:
On Friday, June 29, 2012 9:56:04 AM UTC-5, David Schmitt wrote:
Indeed! Even more so as using the word "declare" for "include" is a
load
of utter bollocks. "Declare" in the common programming language sense
means "indicating the existence
On Tuesday, July 3, 2012 at 8:39 AM, jcbollinger wrote:
> You mean "less objectionable" I presume? I agree that classes generally
> should not define other classes, but the guide already says that elsewhere.
> I guess there's no harm in repeating it, if you think it worth the emphasis.
Sorry,
On Tuesday, July 3, 2012 6:34:10 AM UTC-5, Eric Shamow wrote:
>
> On Friday, June 29, 2012 at 9:55 AM, jcbollinger wrote:
>
> Note, however, that elsewhere the style guide (version 1.1.2) says "*Classes
> should generally not declare other classes*," which is a bunch of
> bologna. A Puppetlabs
On Friday, June 29, 2012 at 9:55 AM, jcbollinger wrote:
> Note, however, that elsewhere the style guide (version 1.1.2) says "Classes
> should generally not declare other classes," which is a bunch of bologna. A
> Puppetlabs employee told me recently that he would have that removed, but
> evide
On Friday, June 29, 2012 9:56:04 AM UTC-5, David Schmitt wrote:
>
> Indeed! Even more so as using the word "declare" for "include" is a load
> of utter bollocks. "Declare" in the common programming language sense
> means "indicating the existence of a thing that is defined elsewhere."
> Like "
On 29.06.2012 15:55, jcbollinger wrote:
On Friday, June 29, 2012 1:51:52 AM UTC-5, Brian Gupta wrote:
According to the style guide, "Classes and defined resource types
must not be defined within other classes."
However looking at
https://github.com/puppetlabs/puppetlabs-nginx
On Friday, June 29, 2012 1:51:52 AM UTC-5, Brian Gupta wrote:
>
> According to the style guide, "Classes and defined resource types
> must not be defined within other classes."
>
> However looking at
>
> https://github.com/puppetlabs/puppetlabs-nginx/blob/master/manifests/init.pp
> shows tha
