On Wed, Apr 22, 2026 at 9:27 PM Amit Kapila <[email protected]> wrote:
>
> On Tue, Apr 21, 2026 at 12:16 PM Peter Smith <[email protected]> wrote:
> >
> > PSA v4, now combined as Vignesh suggested, and with the extra space also 
> > fixed.
> >
>
> I would like to explicitly mention in the commit message that "Tag for
> included publications was a pre-existing behavior but it is better to
> change it for making it consistent with excluded publications tag. So,
> how about a commit message as follows: "psql: Improve describe footer
> titles for publications.
>
> The psql describe (`\d`) footer titles were previously unintuitive
> when listing publications that included or excluded specific tables.
> Even though the tag for included publications was pre-existing, it is
> better to update it to "Included in publications:" to match the
> phrasing of the "Excluded from publications:" tag.
>
> Footer titles for sequence and schema descriptions have been updated
> similarly to maintain consistency."
>
> --

OK, but the order what happened was a bit different.

1/ Firstly, it was only the "Except publications:" footer for \d that
was unintuitive. So that was updated to "Excluded from publications:".
2/ The pre-existing "Publications:" was intuitive as-is, but it was
updated to better match wording of 1.
3/ Then footer titles for sequence/schema were updated same

So maybe reword it slightly something like below:

SUGGESTION

The psql describe (`\d`) footer title was previously unintuitive when listing
publications that excluded specific tables. This has been fixed to say
"Excluded from publications:".

The pre-existing footer title for listing publications that include specific
tables has also been modified to to better match that phrasing. Now it says
"Included in publications:".

Footer titles for sequence and schema descriptions have been updated
similarly to maintain consistency.

======
Kind Regards,
Peter Smith.
Fujitsu Australia


Reply via email to