On 03/25/19 18:03, Ryan Lambert wrote:
> The following review has been posted through the commitfest application:
> make installcheck-world: tested, passed
> Implements feature: tested, passed
> Spec compliant: not tested
> Documentation: not tested
Hi,
Thanks for the review! Yes, that part of this commitfest entry has been
committed already and will appear in the next minor releases of those
branches.
That leaves only one patch in this commitfest entry that is still in
need of review, namely the update to the documentation.
If you happened to feel moved to look over a documentation patch, that
would be what this CF entry most needs in the waning days of the commitfest.
There seem to be community members reluctant to review it because of not
feeling sufficiently expert in XML to scrutinize every technical detail,
but there are other valuable angles for documentation review. (And the
reason there *is* a documentation patch is the plentiful room for
improvement in the documentation that's already there, so as far as
reviewing goes, the old yarn about the two guys, the running shoes, and
the bear comes to mind.)
I can supply pointers to specs, etc., for anyone who does see some technical
details in the patch and has questions about them.
Regards,
-Chap