I was thinking that it would be better to put the `join("\n")` on a separate line so if an error occurred joining items than it would be easier to see that in a stacktrace, but in retrospect, I don't believe that case would ever happen.

I think we are fine to leave it as is.