Hacker News new | past | comments | ask | show | jobs | submit login

Yes, but since then, there has been a continual stream of updates, unit tests (shockingly, the original version had ZERO unit tests), expanded support for a variety of IDEs, and better code structure.

Also, they author finally got around to actually providing a proper pom.xml file.

Check the commit history for details.

If you're interested in contributing, there's definitely room for expanding the unit test coverage, and also instrumenting the build process to support a code coverage tool such as EMMA or Coberture. Also, I don't believe that the code has undergone a security audit, and I don't see any code reviews for many of the commits, which is distressing.

And documentation! The code does not support javadoc, and in fact, there are only two comments in the entirety of the nearly 1500 line code base!




> And documentation! The code does not support javadoc, and in fact, there are only two comments in the entirety of the nearly 1500 line code base!

This is just _so_ non-compliant with Best Practices that it reveals itself too easily as a parody.

Each class should at least have the default Eclipse comment telling you how to change the template for a new class file, and most methods should have auto-generated JavaDoc for the first version of the method which was written years ago, with nothing filled in.




Guidelines | FAQ | Lists | API | Security | Legal | Apply to YC | Contact

Search: