docs: Introduce conventional commits to CONTRIBUTING.md

[xezon]

Merge with Rebase

This change suggests to change commit title tags from our current custom ones to the conventional (extended) commit tags as highlighted by @aliendroid1.

The [GEN][ZH] tags are sort of obsolete anyway because we always replicate the changes to Generals in the same commit anyway. Originally it was intended to not lose oversight over which commits need replicating to Generals.

I also renamed @compile to @build in code comments so that it is consistent with the conventional commit type name.

[Skyaero42]

CI should be changed too, to verify correct use of conventional commits

[xezon]

CI should be changed too, to verify correct use of conventional commits

Yes. I do not know how to do that. It requires changes in validate-pull-request.yml

[xezon]

Regarding the commit tags, how about we begin them with uppercase? All lower case looks strange to me.

[xezon]

I think the lower case sentencing looks strange. Every other title sentence we begin with upper. Every English sentence in books and articles begin with upper case letter. I have no clue why programmers suddently want to begin sentences with lower case.

I will be changing this. The title tags themselves can remain lower case, as we have them elsewhere in texts as well.

[barefootlogician]

I think the lower case sentencing looks strange. Every other title sentence we begin with upper. Every English sentence in books and articles begin with upper case letter. I have no clue why programmers suddently want to begin sentences with lower case.

I will be changing this. The title tags themselves can remain lower case, as we have them elsewhere in texts as well.

Personally I prefer lowercase just because that way its consistent with other repos that use conventional commits however this repo has enforced uppercase for commit headings so far so uppercase is a reasonable choice too as it maintains consistency with past commits.

I do find it odd too though that the convention is lowercase. I think the reasoning was to make it similar to git's messages, same as why imperative mode is preffered. Though imperative mood makes more sense to me because it also has the benefit of using less characters.

[barefootlogician]

CI should be changed too, to verify correct use of conventional commits

Yes. I do not know how to do that. It requires changes in validate-pull-request.yml

There are a few workflow options in github marketplace that we could consider using

[xezon]

@tintinhamans Can you help with the validate-pull-request.yml ?

[Mauller]

looks good to me

[xezon]

Rebased.