#shy-lizard npm-modules #3
Conversation
implementing remaining functions
HJBowers
left a comment
HJBowers
left a comment
There was a problem hiding this comment.
It looks like you started setting up this project like a full stack, including routes and views, although your specs don't require that. In fact, your specs are completely empty. From my memory of doing this project in the past, I think it just requires you to build tests and source files. So, using express is unnecessary.
It looks like your source files are well written, and it's good that you started implementing various methods. In the future, I would recommend trying to build these out without using any methods (this makes some of the algorithms significantly more difficult). I'd also recommend attempting some of the more difficult algorithms if you decide to do this project again.
Over all, your code looks good, and your formatting is clean. Your tests also look good, although it would be useful to have multiple cases to test. If this was your first exposure to testing, then you're doing great! :)
No description provided.