ding2tal/bpi-client — pull request #1 by kasperg (2014-12-17 11:54:51+00:00)
Title: Inital review

I have performed an initial review of the code.

In general the library seems well implemented using a solid foundation, [Goutte](https://github.com/FriendsOfPHP/Goutte), and on the surface of it the implementation seems solid.

My primary concern is that the classes lack a general description of their purpose. This makes it difficult to determine whether the architecture is sound. I would prefer is this could be added and then I would like to take a second look.

The state of dependencies seems messy.

1. I have removed `doctrine/common`. It was not used.
2. I do not like the fact that the library is pinned to Goutte 1.0.1. I have tried changing this to ~1.0 as Goutte uses semantic versioning. This makes the unit tests fail. I have also tried just redownloading the current dependencies. This makes the unit tests fail as well. To me these are signs that something is amiss here.