|
|
|
|
|
by gus_massa
1513 days ago
|
|
I don't mind it's a bot, and I really appreciate that apparently a human made the final review before sending the PR. But I don't like that the tittle of the commit is: > Fix issue probably-meant-fstring found at https://codereview.doctor I expect a more neutral title for a commit, something like > Fix fstring in <name-of-file> Each maintainer/project has their own (weird) rules about titles, and if any other files must log the changes, and regression test, and whatever they like. But I think no maintainer/project expect to see the name of the author in the commit tile. |
|