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
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) |