All the changes for the service browser look fine to me (I integrated
them with my version of the browser and all runs fine).

OK, thanks.

I assume the diff you attached for the issues RIVER-224, 244, 210 and
217 is all the same.

The patch for 224 is separate from the single patch for 210/217/244.
I tried to indicate the multi-issue patch in the comments.

Just to nit-pick, I would prefer only one patch
that mentions that it incorporates multiple issues instead of giving
each issue the same patch.

Where should I have put that one patch, and how should I have referred
to it from the issues?

- Bob

Reply via email to