Re: Build the docs if there are changes in docs and don't run other tasks if the changes are only in docs

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-25 11:56:28
Message-ID: CAN55FZ1b0s9rsTahmWgG+ddivp8FqkpTy0U67Pz9CEg2naTA-Q@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Hi,

I attached the second version of the patch.

On Mon, 11 Sept 2023 at 15:11, Daniel Gustafsson <daniel(at)yesql(dot)se> wrote:
>
> > On 11 Sep 2023, at 13:03, Nazir Bilal Yavuz <byavuz81(at)gmail(dot)com> wrote:
>
> >> 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?
>
> I believe so, the list of scripts and input files can be teased out of the
> doc/src/sgml/meson.build file.

Now the files mentioned in the doc/src/sgml/meson.build file will
trigger building the docs task. Also, if the changes are only in the
README files, CI will be skipped completely.

Any kind of feedback would be appreciated.

Regards,
Nazir Bilal Yavuz
Microsoft

Attachment Content-Type Size
v2-0002-Skip-CI-if-changes-are-only-in-docs-or-in-the-REA.patch text/x-diff 1.8 KB
v2-0001-Only-built-the-docs-if-there-are-changes-are-in-t.patch text/x-diff 3.1 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Karl O. Pinc 2023-09-25 12:00:49 Re: Various small doc improvements; plpgsql, schemas, permissions, oidvector
Previous Message Ashutosh Bapat 2023-09-25 11:23:42 Re: Avoid a possible out-of-bounds access (src/backend/optimizer/util/relnode.c)