Issue #6844 has been updated by James Turnbull. Category set to plumbing Status changed from Accepted to Ready For Testing Assignee set to Nigel Kersten Target version set to 2.6.8 Affected Puppet version set to 2.6.7 Branch changed from next to 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: Ready For Testing Priority: Normal Assignee: Nigel Kersten Category: plumbing Target version: 2.6.8 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.
