The method intent is good, but I suggest to change the method name cause this method can be accessed from outside the class and it will seems weird:
e.g.
```ruby
site = Site.new
site.each do |item|
  # code omitted
end
```

```each``` what? What kind of item a site have? Seems a little confused, can be many different things

I suggest to call it ```each_file```