Re: [Puppet Users] Style Guide question

2012-07-03 Thread Nan Liu
On Tue, Jul 3, 2012 at 7:53 AM, llow...@oreillyauto.com wrote: > Is there an ETA for getting the Style Guide up to date for 2.7.x or even > 3.X? > > Currently it is geared towards the 2.6.X line, and says that certain things > should not be done due to 2.6.x compatibility issues. > > For example,

[Puppet Users] Style Guide question

2012-07-03 Thread llow...@oreillyauto.com
Is there an ETA for getting the Style Guide up to date for 2.7.x or even 3.X? Currently it is geared towards the 2.6.X line, and says that certain things should not be done due to 2.6.x compatibility issues. For example, it says to use this: class ntp( $server = 'UNSET' ) {

Re: [Puppet Users] Style guide question..

2012-06-29 Thread David Schmitt
On 29.06.2012 10:45, Brian Gupta wrote: On Fri, Jun 29, 2012 at 3:09 AM, David Schmitt wrote: On 29.06.2012 08:51, 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/puppetlab

Re: [Puppet Users] Style guide question..

2012-06-29 Thread Brian Gupta
On Fri, Jun 29, 2012 at 3:09 AM, David Schmitt wrote: > On 29.06.2012 08:51, 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/

Re: [Puppet Users] Style guide question..

2012-06-29 Thread David Schmitt
On 29.06.2012 08:51, 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 that there are a number of other classes def

[Puppet Users] Style guide question..

2012-06-28 Thread Brian Gupta
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 that there are a number of other classes defined in class nginx. What's the correct p