Re: Proposal to have a script to check MR for common mistakes

On 12/8/26 07:43, Albert Astals Cid wrote:
> El dimarts, 11 d’agost del 2026, a les 17:45:10 (Hora d’estiu d’Europa
> central), Méven va escriure:
>> Hi everyone,
>>
>> I'd like to add a script to our ci-utilities to allow failure builds based
>> on regular mistakes that regular contributors or newcomers can make.
>>
>> Like:
>>   - putting links to invent.kde.org in commits
>>   - having \since but the incorrect version
>>   - missing \since on new API
>>   - having links to bugs, instead of using our BUG:/CCBUG:
>>
>> I have implemented them:
>> https://invent.kde.org/sysadmin/ci-utilities/-/merge_requests/700
>>
>> And I am thinking about another one:
>>   - prevent commit message prefix with "fix:" or "feature"
>>
>> As a dolphin maintainer and kio one, I really would enjoy those as they
>> will reduce the number of times I have to repeat myself.
>>
>> All of those would be opt-in, not all projects have such preferences.
>> This would be similar to our clang-format check
>> https://invent.kde.org/sysadmin/ci-utilities/-/blob/master/run-clang-format.
>> py?ref_type=heads that a few projects have adopted (dolphin, Kwin)
>>
>> Later I would like to add to those build failures comments on MR with
>> friendly instructions on how to fix them, or having suggestions, like the
>> expected version for \since.
>>
>> Many FOSS projects have such bots/checks, It would be great to add that to
>> lessen the reviewer burden while upholding best practices.
>>
>> What do you think?
> In general, things that save time both to the reviewer and the reviewee are a
> good thing (i.e. better to have a bot saying that since is missing than a
> human, the bot will be faster).

Agreed, anything that can save time via automation and make contributors 
lives easier is a +1 from me.

Regards,

Justin