Hornswaggler/fuzzy-octo-shame — pull request #4 by virtuoushub (2014-11-01 07:58:27+00:00)
Title: Version 0.1.1 Pull Request  

I ran FindBugs and CodePro Analytix, to analyze the code. From the static code analysis findings, I implemented some minor fixes, and documented TODOs/FIXMEs where appropriate.

The biggest change is that I added [Guice](https://github.com/google/guice) as a dependency. [MotherBrain.java](https://github.com/Hornswaggler/fuzzy-octo-shame/blob/master/src/main/java/com/jed/core/MotherBrain.java), now uses dependency injection to [create a singleton](https://github.com/virtuoushub/fuzzy-octo-shame/blob/feature/guice/src/main/java/com/jed/core/MotherBrain.java#L41-44), instead of your [getInstance()](https://github.com/Hornswaggler/fuzzy-octo-shame/blob/master/src/main/java/com/jed/core/MotherBrain.java#L43-45) method. The addition of a dependency injection framework will allow for easier unit testing and more maintainable code.

This is pretty much the same as my other [premature pull request](https://github.com/Hornswaggler/fuzzy-octo-shame/pull/2) with the addition of some extra maven plugins to package the jar with dependencies.