From: | Nazir Bilal Yavuz <byavuz81(at)gmail(dot)com> |
---|---|
To: | Daniel Gustafsson <daniel(at)yesql(dot)se> |
Cc: | PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Build the docs if there are changes in docs and don't run other tasks if the changes are only in docs |
Date: | 2023-09-11 11:03:13 |
Message-ID: | CAN55FZ3KJBBonoqe4mrJUSCz0XyoEaLzjDwGLE+j9ihwsBwD0w@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Hi,
Thanks for the reply!
On Fri, 8 Sept 2023 at 11:05, Daniel Gustafsson <daniel(at)yesql(dot)se> wrote:
>
> > On 7 Sep 2023, at 18:06, Nazir Bilal Yavuz <byavuz81(at)gmail(dot)com> wrote:
>
> > if the changes are only in the docs, don't run
> > all tasks except building the docs task; this could help to save more
> > CI times.
>
> A related idea for docs in order to save CI time: if the changes are only in
> internal docs, ie README files, then don't run any tasks at all. Looking at
> src/backend/parser/README the last two commits only touched that file, and
> while such patches might not be all that common, spending precious CI resources
> on them seems silly if we can avoid it.
>
> It doesn't have to be included in this, just wanted to bring it up as it's
> related.
I liked the idea, I am planning to edit the 0002 patch. CI won't run
any tasks if the changes are only in the README files.
> Almost, but not entirely. There are a set of scripts which generate content
> for the docs based on files in src/, like src/backend/catalog/sql_features.txt
> and src/include/parser/kwlist.h. If those source files change, or their
> scripts, it would be helpful to build docs.
Thanks! Are these the only files that are not in the doc subfolders
but effect docs?
Regards,
Nazir Bilal Yavuz
Microsoft
From | Date | Subject | |
---|---|---|---|
Next Message | Jelte Fennema | 2023-09-11 11:23:54 | Re: proposal: psql: show current user in prompt |
Previous Message | John Naylor | 2023-09-11 11:02:18 | Re: generate syscache info automatically |