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 по дате отправления: