Обсуждение: [PATCH] In psql \?, add [+] annotation where appropriate

Поиск
Список
Период
Сортировка

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

От
"Matthijs van der Vleuten"
Дата:
Hi,

User inoas on IRC channel #postgresql noted that \? does not describe \do as supporting the + option. It does however
supportthis 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
ispreserved 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
Вложения

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

От
Neil Chen
Дата:
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.

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

От
Michael Paquier
Дата:
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

Вложения