octokit/octokit.net — pull request #298 by pmacn (2014-01-11 07:39:37+00:00)
Title: Implementing Deployments client; Issue #297

I'm having some doubts about a few naming things still, especially some inconsistencies regarding status/statuses. Just something to keep an eye out for and I wouldn't mind a comment or two on that if someone has an opinion about it.

There is also a huge chunk of duplicated code between the two integration tests classes, not to mention that a lot of that came from an already existing integration test. It's still DRY if I'm repeating someone else's code, right?

Anyway, I'm sure I could put some effort into cleaning that up and maybe implement some sort of setup "builder" if that's something we'd want to do. That could hopefully replace the setup in the other integration tests as well, making them all more declarative.

Other than that I hope that this should be satisfactory for the most part; After all it was a rather straight forward implementation.

If I have time I'll get the Reactive one thrown together tomorrow unless there's a whole horde of issues to resolve with this one.