OpenSSL Communities

PR titles and commit messages

Paul DalePaul Dale Tue 17 Feb 2026 11:05PMPublicSeen by 101

This is aimed at the people here who contribute to the OpenSSL code base.

Recently there have been a flurry of PR's with distracting or useless titles.
These are often associated with commit messages that are also inappropriate.

For example #30050 has a title of "Cleanup required in Aisle-1" and a commit message of "aisle-1". Both contain no relevant information for the viewer. I'd go as far as saying they are worse than useless since they are liable to waste the viewer's time.

Could we please include at least some relevant information in the title and commit messages and stay professional in the commit messages.

Readers don't care about your dog's name, car preferences, Internet memes and the like.

That the project's staff are also culprits here disappoints me.

Todd Short

Todd ShortWed 18 Feb 2026 12:04AM

Fully agreed. At first I thought it was a typo of the username, but it appears that there was a typo of the username in the commit message.

Shane Lontis

Shane LontisWed 18 Feb 2026 1:16AM

Agree also.

Tom Cosgrove

Tom CosgroveWed 18 Feb 2026 8:49AM

I've commented in #30050 as well, since the change only updated the PR title, and not the commit message, which is IMO even more important. Generally, to make things easier for people wanting to interact with OpenSSL as a project, PRs should stand alone, not require clicking through to figure out what they're doing, and commits should stand alone so that someone reading the git history can see what's been happening without going to PRs and issues for context (or requiring inside knowledge).

Tomas Mraz

Tomas MrazWed 18 Feb 2026 9:22AM

I'd suggest updating `CONTRIBUTING.md` and `.github/PULL_REQUEST_TEMPLATE.md`

Paul Dale

Paul DaleThu 19 Feb 2026 12:06AM

I've raised a PR but this should be second nature to any software engineer.
Especially so for the project's staff.