Skip to content

Conversation

@phroa
Copy link

@phroa phroa commented Dec 13, 2015

I'm super unfamiliar with this library. This PR probably needs a load of review; I just wanted it to compile on the latest version so I can use it in a project, which it does.

I tried to go through and fix the build warnings that I felt comfortable fixing, but I don't know exactly what the state of the warnings were beforehand since it wouldn't compile. By no means did I remove them all, though :)

Tests fail, but given the fact that it can't find assertEquals leads me to believe that they weren't always working. I didn't even try to fix them. Here's the output of gradle assemble test. https://gist.github.com/phroa/31c18af5c25c2f29a47a


Comments appreciated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant