Kevin-Rau / breakfast

0 stars 4 forks source link

Nathan Lapinski Review #11

Open nathan-lapinski opened 9 years ago

nathan-lapinski commented 9 years ago

A good first shot at a RESTful Node service. Couple of points that others have mentioned - use Mongoose for connecting to the db. Also, consider documenting things a little more professionally as this is a public repo (doStuff.js, for example, reads better as server.js. A readme that describes how to set up and run everything would also be in order).

I had some troubles getting it to run, but reading through the router code, it looks like everything should function. The code is also pretty clean and easy to read - so nice work there!

Overall, I thought this was a good project.