Re: Proposal to include --exclude-extension Flag in pg_dump

> I ran it through pgindent to fix some whitespace issues and added
> another test for the filter option, based on the test case you added.

Thank you for addressing those whitespaces issues and adding more tests. I
appreciate your attention to detail and will certainly be more vigilant in
future.

> I'm marking this ready-for-commit (which I'll probably do myself in a
> day or two, unless anyone else claims it first).

Thank you very much, this marks my first contribution to the open-source
community, and I'm enthusiastic about making further meaningful
contributions to PostgreSQL in the future.

> LGTM too from a read through. I did notice a few mistakes in the --filter
> documentation portion for other keywords but that's unrelated to this
patch,
> will fix them once this is in to avoid conflicts.

Thanks Daniel for your review. It's gratifying to see that my patch not
only introduced the intended feature but also brought other minor mistakes
to light.

Regards
Ayush Vatsa
Amazon Web services (AWS)

On Tue, 19 Mar 2024 at 17:23, Daniel Gustafsson <daniel(at)yesql(dot)se> wrote:

> > On 19 Mar 2024, at 12:19, Dean Rasheed <dean(dot)a(dot)rasheed(at)gmail(dot)com> wrote:
>
> > I'm marking this ready-for-commit (which I'll probably do myself in a
> > day or two, unless anyone else claims it first).
>
> LGTM too from a read through. I did notice a few mistakes in the --filter
> documentation portion for other keywords but that's unrelated to this
> patch,
> will fix them once this is in to avoid conflicts.
>
> --
> Daniel Gustafsson
>
>