convention schmonvention in this case. :smiley:

I like the memoization idea and just to be clear, I would make that method
private.


On Mon, Aug 19, 2013 at 11:32 AM, Parker Moore <notifications@github.com>wrote:

> In lib/jekyll/site.rb:
>
> >  module Jekyll
> >    class Site
> >      attr_accessor :config, :layouts, :posts, :pages, :static_files,
> >                    :categories, :exclude, :include, :source, :dest, :lsi, :pygments,
> >                    :permalink_style, :tags, :time, :future, :safe, :plugins, :limit_posts,
> > -                  :show_drafts, :keep_files, :baseurl
> > +                  :show_drafts, :keep_files, :baseurl, :site_cleaner
>
> It follows convention. We ought not use @site_cleaner. I'd be OK with
>
> def site_cleaner
>   @site_cleaner ||= Jekyll::Site::Cleaner.new(self)end
>
> as an alternative.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/mojombo/jekyll/pull/1429/files#r5846781>
> .
>