I mostly made this change just to hew a bit closer to the solutions. But (although it wasn't formally stated in the solution) I like there being a consistent pattern (with other User methods) where I'm defining the user's attributes such as following_relationships, rather than creating a new Relationship that happens to include the user. I think it makes it a little easier to understand later. Thanks for looking over the code!