jirishus/lendingApp — pull request #1 by francismakes (2014-04-15 21:13:02+00:00)
Title: (review) Refactors main HTML and JS codebase.

Hey Brandon,

I enjoyed going through your application and seeing what I could contribute. I had some trouble reading some of the code, so I refactored it for clarity and readability. I did not change the core of your code as everything works the same. I have a different style for writing JS, so some of it may look fancy or a bit much. Take what you need, and let me know if you are working on adding any features to your app in the next week.

Quick questions:

* __Do you use Grunt.js for this project's development?__ If you already do, I think it would be awesome if you put your `Gruntfile.js` or `Gruntfile.coffee` for future development. I could also learn from your config settings.
* __Have you considered using jshint?__ If you already do, please upload that! I would like to respect your project's style guide and write code according to your settings. In my projects, I use [gulp-jshint](https://github.com/spenceralger/gulp-jshint) and [jshint-stylish](https://github.com/sindresorhus/jshint-stylish) to quickly debug any JS code. I also find [precommit-hook](https://github.com/nlf/precommit-hook) as a helpful NPM that checks your code against jshint before committing to `git`.
* __Have you considered adding a package.json__? If you scale your application with more features or packages, it would be helpful to have this in the directory.

All optional suggestions alongside questions, but let me know what you think!