Issue #6844 has been updated by Max Martin. Status changed from Ready For Testing to Closed
Merged into 2.6.next in commit:2892068 ---------------------------------------- Bug #6844: ./puppet.rb has a single case statement in self.[](param) https://projects.puppetlabs.com/issues/6844 Author: ben hengst Status: Closed Priority: Normal Assignee: Nigel Kersten Category: plumbing Target version: 2.6.9 Affected Puppet version: 2.6.7 Keywords: Branch: https://github.com/notbenh/puppet/commit/c29cf8f075ad4f2258e9acf26c6d27cd9d21b352 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.
