Re: Include patch id in commitfest page
От | Magnus Hagander |
---|---|
Тема | Re: Include patch id in commitfest page |
Дата | |
Msg-id | CABUevEyd40TxEfAG3m_b1-qOt+YK0a8wPrwCUR_aTqkOm23xFA@mail.gmail.com обсуждение исходный текст |
Ответ на | Re: Include patch id in commitfest page (Peter Eisentraut <peter@eisentraut.org>) |
Список | pgsql-hackers |
On Sat, Jan 11, 2025 at 10:40 AM Peter Eisentraut <peter@eisentraut.org> wrote:
I want to register that I'm not a fan of this change:
https://git.postgresql.org/gitweb/?p=pgcommitfest2.git;a=commit;h=eee60a5332ac0745071086477ff4e189b080d3ab
Rationale: It puts the least important information first. For the most
part, the CF entry number is irrelevant, so it shouldn't be shown in
such a prominent position.
Maybe
<a>title here</a> (number here)
would be a compromise. But looking at the original argument:
"""
Clicking the patch header would sort by the patch ID, but that ID would
not actually be visible. Effectively resulting in random ordering for
the user, so let's just show the ID.
"""
I would suggest that the actual fix would be to order by title instead
of ID.
Or just turn the sorting off.
I'll revert this one pending further discussions, and then we can figure out what we want to put back.
В списке pgsql-hackers по дате отправления: