Horton

Thread

Commits

Same data as JSON: GET /api/v1/messages/:b64id/commits the thread's linked commits as JSON, with link sources. API reference →

  1. Fix inconsistencies in psql --help=commands

  1. [PATCH] In psql \?, add [+] annotation where appropriate

    Matthijs van der Vleuten <postgresql@zr40.nl> — 2021-03-21T19:41:47Z

    Hi,

User inoas on IRC channel #postgresql noted that \? does not describe \do as supporting the + option. It does however support this option, as do \dAp and \dy.

This patch adds the annotation to the description of these commands in \?.

While adding it to the translation files I noticed some obvious errors, so this patch fixes the following as well:

* correct inconsistent alignment. This was especially egregious in fr.po. Differing alignment across sections in es.po is preserved as this may have been a deliberate choice by the translator.
* cs.po: remove extraneous newline
* es.po: replace incorrect mention of \do with the correct \dAc, \dAf and \dAo
* fr.po: merge some small lines that still fit within 78 chars
* fr.po: remove [brackets] denoting optional parameters, when these aren't present in the English text
* fr.po: \sv was shown as accepting "FONCTION". This is replaced with "VUE".
* fr.po: \t was missing its optional [on|off] parameter.
* tr.po: fix typo at \d
* uk.po: add missing newline at \c

Regards,
Matthijs

  2. Re: [PATCH] In psql \?, add [+] annotation where appropriate

    Neil Chen <carpenter.nail.cz@gmail.com> — 2021-05-25T06:10:15Z

    The following review has been posted through the commitfest application:
make installcheck-world:  tested, passed
Implements feature:       tested, passed
Spec compliant:           tested, passed
Documentation:            tested, passed

Hi, thank you for your work. I think this is a meaningful patch that should be merged.

  3. Re: [PATCH] In psql \?, add [+] annotation where appropriate

    Michael Paquier <michael@paquier.xyz> — 2021-06-09T07:29:09Z

    On Tue, May 25, 2021 at 06:10:15AM +0000, Neil Chen wrote:
> Hi, thank you for your work. I think this is a meaningful patch that
> should be merged.

Merged, then.  I have scanned the rest of the area and did not notice
any other inconsistencies.
--
Michael