Hi!

To summarize here:

- Good git commit message titles should NOT end with a dot. Titles
(and e.g. subject lines in an email) should not end with a dot based
on general English grammar rules and also software development / git
conventions as long as we have had patches and git commits.

- By convention, many people use full sentences in changelogs (as
pointed out by e.g. Mattia in
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=959665). In a
changelog it makes sense to have full sentences if the texts are
longer. This is also what English grammar rules say: a (bullet point)
list with just titles are not sentences and have no trailing dot, but
if the list has longer and *some* items that are sentences, then all
items in the list should end with a dot.

Therefore, can we please make sure that Lintian-brush and things that
do git commits do NOT add trailing space, and that tools that write
changelogs add trailing dots when needed?

I think Lintian Janitor is great and I love raising the bar on
quality, but I am allergic to git commit titles and subject lines that
have a trailing dot.

Currently I want to support Janitor, but at the same time I don't want
it to pollute my git history and suddenly have titles with trailing
dot visible in e.g.

$ git log --oneline
af616d4bb4a Salsa-CI: Extend tests to install and run Cacti as extra validation
abcdabcd Bump debhelper from old 12 to 13.
a3b6f3d7f4b Increase MTR verbosity and unify skip test lists with how
d/rules does it
fdcc4c7493f Packages mariadb-plugin-provider-* must depend at least on
MariaDB 10.11
44ae54b6dca Salsa-CI: Use pip3 with --break-system-packages for Python
3.11 compatibility
5a2361935e8 Update Brazilian translation of debconf messages (Closes: #1030908)
cc86e20577c Update French translation of debconf messages (Closes: #1030581)
8f87f472749 (tag: debian/1%10.11.1-4) Update changelog for 1:10.11.1-4 release
741c3b3cdb5 Refresh patches and include references to Debian bugs they
should fix
7ac10dee3b9 Add upstream patch to fix build failure on HPPA (Closes: #1006529)
d0aa92192a8 Temporarily disable MTR on s390x as Debian buildd seems unstable
dd25c71d47d Update existing patch with latest upstream PR#2448 version 0284207

Reply via email to