Errant semantic block cases around class mutation

[dnsco]

Given that the point of this block is to add a method to the struct, I think it should be do end.

ValueObject = Struct.new(:events) {
    def random_name
      @event_name ||= events.sample.name
    end
  }

I've also hit this issue a couple other times when doing metaprommey things.

[dnsco]

just saw #109

[jmkoni]

What we would recommend is the following:

class ValueObject < Struct.new(:events)
  def random_name
     @event_name ||= events.sample.name
  end
end