We're in the `Site` object already though. `each_file` may not be very
descriptive, but it's an improvement over the `each` which was there
before, and `each_site_file` seems a bit redundant from my point of view.


On Tue, Jul 9, 2013 at 1:30 PM, Parker Moore <notifications@github.com>wrote:

> In lib/jekyll/site.rb:
>
> > @@ -255,15 +240,7 @@ def cleanup
> >
> >        # files to be written
> >        files = Set.new
> > -      self.posts.each do |post|
> > -        files << post.destination(self.dest)
> > -      end
> > -      self.pages.each do |page|
> > -        files << page.destination(self.dest)
> > -      end
> > -      self.static_files.each do |sf|
> > -        files << sf.destination(self.dest)
> > -      end
> > +      each_file { |item| files << item.destination(self.dest) }
>
> each_site_file? each_file isn't very descriptive... @mattr-<https://github.com/mattr->
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/mojombo/jekyll/pull/1144/files#r5097660>
> .
>