Good point on the naming convention. Must've slipped my mind -- I'll have a patch out for it shortly.
On Mon, Apr 25, 2011 at 12:32 PM, Randall Hansen <[email protected]>wrote: > On Mon, Apr 25, 2011 at 12:20 PM, Pieter van de Bruggen > <[email protected]> wrote: > > > + def inherit_options_from(action) > > + if action.is_a? Symbol > > + name = action > > + action = @face.get_action(name) or > > + raise ArgumentError, "This Face has no action named #{name}!" > > + end > > "Face" shouldn't be user-facing in CLI context. If we need to label > it we should call it by its common name, "subcommand," and we should > provide the name of the subcommand. Not all this output will be > viewed in real-time on a terminal, and users need to know what they > ran without hunting for it. E.g., > > "Puppet: 'node' has no 'something_I_cannot_do' action. See `puppet help > node`." > > See: > https://sites.google.com/a/puppetlabs.com/ux/investigations/cli-principles/help-access-to > > Note to non-Puppet employees: this document isn't public yet, but I'm > working hard to get it published by Puppet Camp later this week. > > r > > -- > You received this message because you are subscribed to the Google Groups > "Puppet Developers" 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-dev?hl=en. > > -- You received this message because you are subscribed to the Google Groups "Puppet Developers" 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-dev?hl=en.
