I thought @AnthonyMastrean wanted to avoid setting Boolean flags. We can initialize it to false by default to make it more explicit in implementation.

On 25/01/2013, at 6:21 PM, Henrik Feldt <notifications@github.com> wrote:

> In lib/albacore/output.rb:
> 
> > @@ -31,9 +31,9 @@ def erb(f, ft, locals)
> >      File.open("#{@dir_to}/#{ft}", 'w') {|f| f.write erb.result(ErbBinding.new(locals).get_binding)}
> >    end
> >    
> > -  def self.output_to(dir_to, dir_from)
> > -    FileUtils.rmtree dir_to
> > -    FileUtils.mkdir_p dir_to
> > +  def self.output_to(dir_to, dir_from, keep_to)
> > +    FileUtils.rmtree dir_to unless keep_to
> false by default because it's not defined - @AnthonyMastrean - what do you think of this coding pattern? I think we could at least have a attr_accessor :keep_to, remove the method, add rdoc on top of the attr_accessor and default it to false (instead of undefined)
> 
> —
> Reply to this email directly or view it on GitHub.
>