Issue #6844 has been updated by ben hengst.

% Done changed from 0 to 100

Change that I am proposing: 
https://github.com/notbenh/puppet/commit/c29cf8f075ad4f2258e9acf26c6d27cd9d21b352


----------------------------------------
Bug #6844: ./puppet.rb has a single case statement in self.[](param) 
https://projects.puppetlabs.com/issues/6844

Author: ben hengst
Status: Unreviewed
Priority: Normal
Assignee: 
Category: 
Target version: 
Affected Puppet version: 
Keywords: 
Branch: next


The case statement in question only has two options and it seems needless when 
the rest of the file uses if/else for this type of action. It would be more 
consistent to just change this to be an if/else statement like the rest of the 
methods.

Version: I am looking at the next branch though it seems to be like this 
everywhere that I've looked thus far.
Error: none, this is just a code style gripe.
OS: I am running this on Ubuntu 10.10, though this is platform independent.
Ruby: 1.8.7 though again this is platform independent.



-- 
You have received this notification because you have either subscribed to it, 
or are involved in it.
To change your notification preferences, please click here: 
http://projects.puppetlabs.com/my/account

-- 
You received this message because you are subscribed to the Google Groups 
"Puppet Bugs" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/puppet-bugs?hl=en.

Reply via email to