EFregnan

Results 8 comments of EFregnan

It seems there was a bit of discrepancy between the name of the class reported in the documentation and the real name of the class. The class we need is...

@uwolfer, I started to work on this but I soon realised that the amount of classes and dependencies we need to add is quite big. To complicate the situation further,...

For instance, they often use classes from the _AutoValue Annotations_ library. I also found some uses of classes from _Jgit_. Some other external dependencies are used, but generally just once...

Hi, thank you for pointing this out. The files in the src/main/java/com/google directory are periodically synched with the ones of the main Gerrit repo. For this reason, you should first...

Sure, I will create a first "cluster" of parsers and open a PR so you can have a look.

I added a PR with an example of clusters. I think that proceeding in a similar fashion we could cut the number of parameters in the constructors of _ChangeApiRestClient_ or...

@uwolfer, I would like to add a folder _parsers_ in _gerrit-rest-java-client/src/main/java/com/urswolfer/gerrit/client/rest/http/changes/_ to separate parser classes from the other classes (e.g., ChangeApiRestClient.java). What do you think?