[Engines Guide] Remove to_s example, since it's outside the scope

[cllns]

Overloading the to_s method is outside the scope of engines.

By calling .name on author, we can avoid talking about it.

[rails-bot]

r? @eileencodes

(@rails-bot has picked a reviewer for you, use r? to override)

[eileencodes]

Thanks for the PR! 😄

While perhaps it's out of scope of engines I still think this is useful information for beginners. I don't agree with removing it. Why do you think it should be removed?

[cllns]

It's definitely useful information, and I agree it should be in the guides.
Engines aren't trivial, so I don't think they're a place for extraneous information.

I feel like if you're looking into Engines, you likely already know this.
If a beginner is wondering why they're getting something like #<User:0x00000100ccb3b0>, they might look in documentation for views instead.

Maybe the bottom of section 3.1.1 of the Action View Guide might be a better place for this, as a TIP?
http://edgeguides.rubyonrails.org/action_view_overview.html#templates

What do you think of that?

[eileencodes]

Yea that looks like it could be a good place. 👍

[cllns]

What about my most recent push?

I would've copied the whole text, but I feel like reducing it to a TIP is better, considering the section it's in.

[eileencodes]

I'm sorry I didn't read it closely. I agree that's a weird thing to say - to override the to_s in the engine. I've been immersed in sprockets and I think it's making my brain mushy.

Can you remove the tip? I know I asked you to add it but I wasn't thinking. Your change was good originally.

[cllns]

No problem! All set.