| From: | "Karl O(dot) Pinc" <kop(at)karlpinc(dot)com> |
|---|---|
| To: | Brar Piening <brar(at)gmx(dot)de> |
| Cc: | vignesh C <vignesh21(at)gmail(dot)com>, Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org>, Ian Lawrence Barwick <barwick(at)gmail(dot)com>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: doc: add missing "id" attributes to extension packaging page |
| Date: | 2023-01-09 15:53:38 |
| Message-ID: | 20230109095338.38d9bf97@slate.karlpinc.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Mon, 9 Jan 2023 08:09:02 +0100
Brar Piening <brar(at)gmx(dot)de> wrote:
> On 09.01.2023 at 03:31, vignesh C wrote:
> > The patch does not apply on top of HEAD as in [1], please post a
> > rebased patch:
> This one applies on top of 3c569049b7b502bb4952483d19ce622ff0af5fd6
> and the documentation build succeeds. Beyond rebasing I've added a
> few more ids (to make the other patch
> (make_html_ids_discoverable.patch) build without warnings again) but
> nothing that would justify another review.
Agreed. I believe that as long as your system has xmllint installed
and the documentation builds there's not a lot that can go wrong.
This patch only adds lots-of-id attributes.
> We probably have to move quickly with this patch since it touches
> pretty much any file in the documentation and will be outdated in a
> minute.
+1
Regards,
Karl <kop(at)karlpinc(dot)com>
Free Software: "You don't pay back, you pay forward."
-- Robert A. Heinlein
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Pavel Luzanov | 2023-01-09 16:09:19 | psql: Add role's membership options to the \du+ command |
| Previous Message | Pavel Stehule | 2023-01-09 15:50:29 | Re: PL/pgSQL cursors should get generated portal names by default |