public inbox for [email protected]  

help / color / mirror / Atom feed

From: Christoph Berg <[email protected]>
To: Jonathan S. Katz <[email protected]>
Cc: PostgreSQL WWW <[email protected]>
Cc: PostgreSQL Contributors <[email protected]>
Subject: Re: Rename Contributor to Significant Contributor
Date: Mon, 24 Nov 2025 21:52:09 +0100
Message-ID: <[email protected]> (raw)
In-Reply-To: <[email protected]>
References: <[email protected]>
	<[email protected]>

Re: Jonathan S. Katz
> * The fixtures edits will be purely for testing purposes; those changes will
> need to be added into the "Contributor types" section of the admin panel.

Ah ok. I can see them there, but lack permission to edit.

> * I'd note in the commit message that there are a few other cosmetic changes
> and list them out.

Ok.

> > https://www.postgresql.org/community/contributors/

What I forgot to mention is a small detail that is perhaps
interesting: I'm changing

These are the fine people that make PostgreSQL what it is today!

to

These are the fine people that made PostgreSQL what it is today!
                                 ^

Good/bad?

> > * I added a "PostgreSQL" to the (in-page) title so "PostgreSQL
> >    Contributor Profiles" sounds better than a mundane "Contributor
> >    Profiles" listing
> 
> This is fine; I think it's also OK to add it to the title block, since it'll
> appear in search results with that too, but I'm not wed to that opinion.

The current <title> is already "PostgreSQL: Contributor Profiles", so
duplicating that is a bit weird. (But having header title and in-page
title deviate is also weird...)

> > * I moved the "how to suggest" section to the end so the list isn't
> >    cluttered with too much text at the top
> 
> I'm not sure what the reasoning was to have it originally up top - maybe it
> was to make it easier to find that info, especially because the page is
> fairly long. I'd be OK with keeping it up top - it's not too much to scroll.

IIRC originally there was only some "mail webmaster" bit at the bottom.

I think that page has this information:

1. who are the people on the list
2. how do I add someone

And currently 2 is before 1, which seems weird to me at least.

> LGTM. Thanks!

Thanks!

Christoph

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: [email protected]
  Cc: [email protected], [email protected], [email protected], [email protected]
  Subject: Re: Rename Contributor to Significant Contributor
  In-Reply-To: <[email protected]>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox