No, I put parens only when defining. During calls I put parents around
parameters if there are more than four params.
On Nov 11, 2013 12:08 PM, "jay-altiscale" <notifications@github.com> wrote:

> In lib/commands.rb:
>
> > @@ -62,7 +64,7 @@ def execute(command)
> >        global_known_hosts_file: '/dev/null') do |session|
> >        logger.info "Logged into #{@host} to run #{command}"
> >        start_time = Time.now.to_i
> > -      status = execsh command, session, command
> > +      status = execsh command, session, command, &validator
>
> I see you are following bracket around function parameter, so to be
> consistent, you can follow the same here as well.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/AltiPub/perf-framework/pull/4/files#r7571495>
> .
>