On Mon, Aug 5, 2024, at 10:33 AM, Daniel Gustafsson wrote: >> On 5 Aug 2024, at 13:16, Marcos Pegoraro <mar...@f10.com.br> wrote: >> >> Em seg., 5 de ago. de 2024 às 07:54, jian he <jian.universal...@gmail.com> >> escreveu: >>> >>> [commitId_link1, commitId_link2]: Change functions to use a safe >>> search_path during maintenance operations (Jeff Davis) >> >> I don't like that prefix dirtying each item. > > I too would prefer links at the end, not least since we might have 2 or 3 (or > more) links for an item.
+1. > Python also links to the Github issue and not the commit, in our project the > analog to that would be linking to the thread in the mailinglist archive as > well as the commit. For us linking to the commit is probably preferrable > since > it will link to the thread but the thread often wont link to the commit (like > a > Github issue will). Maybe icons for code/emailthread can be used to make it > clear what the link is, and cut down to horizontal space required? PgBouncer adds the PR number at the end [1] too. I generally prefer this style because you read the message and after that if you want additional detail you can click on the link at the end. I don't know if linking to the thread is a good idea. We have long long threads that cannot provide quick information for the reader. Since we don't have a concept of every commit has a CF entry, IMO we should use only commits here. The commit message points to the discussion so it is a good start point if you want to research about that specific feature. If one commit is not sufficient for that item, we can always add multiple commits as we usually do in the release notes comments. [1] https://www.pgbouncer.org/changelog.html -- Euler Taveira EDB https://www.enterprisedb.com/
