I like this change... But, giving the context of the attributes, ```site_files``` seems a redundant name, since the attribute is a ```Site``` instance attribute... A practice of mine is ever think how the attribute looks like accessing it from out the class. For example:
```ruby
site = Site.new
site.files
```

looks prettier than

```ruby
site = Site.new
site.site_files
```