-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Description
Hi I have downloaded Docker-Java code and I have commented Guava dependency and I have seen that it is only used for two reasons, to generate a toString, for escaping url path. IMHO adding a dependency of 2MB for only these three features it seems a bit big. Maybe removing this big dependency and add a self implementation of both methods would fit better the project. First because of download artifects you require to run it, and second and most important because you are going to avoid conflicts with developers, for example if one project needs an old version of guava and cannot run with the one provided by docker-java or for example because company have a policy to not use guava (which I know some companies who have it).
I can help on this but without removing jersey-guava because it is a dependency that comes from jersey-client.