I think the elephant in the room is:  where is it stated that we want to write purely idiomatic Ruby?

-Josh

On Jun 30, 2013, at 13:48, Luke Imhoff <notifications@github.com> wrote:

> In modules/post/windows/gather/memory_grep.rb:
> 
> > @@ -40,13 +42,13 @@ def run
> >  		regex = Regexp.new(datastore['REGEX'])
> >  		target_pid = client.sys.process[name]
> >  
> > -		print_status("Found #{datastore['PROCESS']} running as pid: #{target_pid}")
> > -
> >  		if not target_pid
> No, it's part of the Ruby Style Guidelines and part of writing idiomatic Ruby.
> 
> —
> Reply to this email directly or view it on GitHub.