agrinb/book_reviews — pull request #1 by hchood (2014-07-07 17:25:41+00:00)
Title: Hh code review

Hey Alek,

The systems check looked generally good. The one major note is not to use scaffolding to build your apps. It's useful for prototyping things quickly, but that's about it. With scaffolds, you end up with lots of unnecessary code and functionality that you don't need, which can cause problems later (if, for example, you give users the ability to delete records by accident). Plus, you're not really using testing to drive the development of your app when you use a scaffold. The idea with testing is that you only write the code you need to make a test pass, resulting in a clean and simple codebase.

There were a few other minor issues -- check the commits in the PR for additional details.

Let me know if you have any questions,

Helen