| From: | David Rowley <dgrowleyml(at)gmail(dot)com> |
|---|---|
| To: | Erik Wienhold <ewie(at)ewie(dot)name> |
| Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Bruce Momjian <bruce(at)momjian(dot)us>, Nathan Bossart <nathandbossart(at)gmail(dot)com>, Euler Taveira <euler(at)eulerto(dot)com>, pgsql-docs(at)lists(dot)postgresql(dot)org |
| Subject: | Re: Use uppercase keywords in foreign key tutorial |
| Date: | 2025-10-31 03:03:08 |
| Message-ID: | CAApHDvoqh_K2KTocwWYrGw5vLubnPnTzd1SvYTTonQ=vUXP7Ug@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-docs |
On Fri, 31 Oct 2025 at 13:53, Erik Wienhold <ewie(at)ewie(dot)name> wrote:
> But I also agree with Tom that keeping a consistent style is impossible
> in the long run. But it also shows that the docs are still written by
> humans. As long as we can keep a consistent style within a single
> listing (or even an entire page) I'm satisfied as a reader.
For me, I don't see this as a reason not to try. If we do get things
to a consistent point, then anyone making changes that reference
existing portions of the documentation for inspiration should maintain
consistency. If we're entirely random, then there's no hope for anyone
to figure out what the best practice or perfected casing is.
> Besides that I've fixed a couple of more places that had lowercase
> keywords or were missing some whitespace that I had missed before.
A couple of things.
1) I see you've added a space after "INSERT INTO table" and before the
column list, but not consistently, per:
git grep -E "INSERT INTO \w+\("
2) An identifier casing has been changed here:
-SELECT sub_part, SUM(quantity) as total_quantity
+SELECT sub_part, sum(quantity) AS total_quantity
You could also look at the results of the SQL command that's returned
by the following SQL to see if there's anything else. I do see some
"ROLLUP(", "EXISTS(", "GROUPING(" and "VALUES(" in there. You have
been changing "VALUES(" to "VALUES (", so I assume those ones have
been missed:
select 'git grep -E "\b(' || string_agg(UPPER(word),'|') || ')\("'
from pg_get_keywords();
David
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Shinya Kato | 2025-10-31 07:13:51 | doc: Clarify ANALYZE VERBOSE output |
| Previous Message | Erik Wienhold | 2025-10-31 00:52:48 | Re: Use uppercase keywords in foreign key tutorial |