Re: Official devcontainer config

From: Jelte Fennema-Nio <postgres(at)jeltef(dot)nl>
To: Junwang Zhao <zhjwpku(at)gmail(dot)com>
Cc: PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>, Andrey Borodin <amborodin86(at)gmail(dot)com>, tristan(at)partin(dot)io
Subject: Re: Official devcontainer config
Date: 2024-08-01 16:29:19
Message-ID: CAGECzQSfrkGFA=ZUVKJE9i-doUQ9a1cPzKjWZHYPd0pSndSZ1A@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Thu, 1 Aug 2024 at 16:56, Junwang Zhao <zhjwpku(at)gmail(dot)com> wrote:
> I post my daily used devcontainer config[2] , Jelte(ccd)
> suggested that it might be a good idea we integrate the
> config into postgres repo so that the barrier to entry for
> new developers will be much lower.

In my experience adding a devcontainer config has definitely made it
easier for people to contribute to Citus. So thank you for working on
this! This is not a full review, but an initial pass.

> After diving into the container, the postCreateCommand.sh
> will be automatically called, it will do some configurations
> like git, perf, .vscode, core_pattern, etc. It also downloads
> michaelpq's pg_plugins and FlameGraph.

I think the .git settings don't fit well here, they are mostly aliases
which are very much based on personal preference and not related to
Postgres development. It seems better recommend users to use the
devcontainer dotfiles support for this:
https://code.visualstudio.com/docs/devcontainers/containers#_personalizing-with-dotfile-repositories

> - pgindent

It might make sense to install Tristan (ccd) his Postgres Hacker
extension for vscode to make this a bit more userfriendly:
https://marketplace.visualstudio.com/items?itemName=tristan957.postgres-hacker

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Andres Freund 2024-08-01 16:44:08 Re: PG17beta2: SMGR: inconsistent type for nblocks
Previous Message Nathan Bossart 2024-08-01 16:21:56 Re: [PATCH] Add crc32(text) & crc32(bytea)