* It is better to be idiomatic.
* Changing things like this could give pull requests friction. Another example would be spaces in empty lines cleanup which some people have git setup to do. Adding that style to a PR just add's a bit of noise.
* The technical reasons for each are a bike shed. However, all of Enumerable is built around the idiomatic `collection.dosomething` syntax. There are no alternatives for for `each_with_index`, `inject`, `map`, `all?`, etc, etc. So the question then becomes, why make `each` different when all other Enumerable options will never look that way.

FWIW, I did change it back :)