Is this really an improvement? Is this a rubocop we want to enforce?

I don't think so. I don't care too strongly, but it seems to me that def self.method is more concise and still unambiguous. Especially with longer class names the 'recommended' way could become unwieldy.

Your answer confuses me, as I'm interepreting the "recommended" way to mean the way rubocop enforces. The "non-recommended" could become "unwieldy" with super-long class names.

Ha - that's because this morning I was reading it totally backwards for some reason and what I said makes no sense. You're right, the non-recommended way could become unwieldy.

I actually do prefer the rubocop recommended self.method over ClassName.method style.

As to whether we enforce it, I don't feel too strongly about it.