Add the logger to the user struct to pass around

[chrisgilmerproj]

I'm not sure why I didn't think of this in my logging PR. By passing logger as part of the User struct we eliminate the need to pass a logger around via functions calls. This really simplifies the code and makes it easier to read.

Also, the more I consider it the User struct is really just the configuration for the setup function. When I think of it in that context then the Logger doesn't feel out of place in the struct. So in a future PR we might want to rename User to SetupConfig.

[eeeady]

It's a little confusing if you leave it as a struct called User but I agree that it's more readable and since you plan on renaming it I'm on board.

[chrisgilmerproj]

It's a little confusing if you leave it as a struct called User but I agree that it's more readable and since you plan on renaming it I'm on board.

I may just make a PR to do the renaming soon. I want to make sure some of these structural changes are done separately from #67.