Nice improvement on the guidelines.
Here is a little feedback compared to utopian V2:
Commits message are irrelevant for us because the pull request is squashed and merged into the develop branch. The Pull request also needs to be rebased from the develop branch which can result in a single commit in the pull request. This is done because the pull request has to contain only the files modified by the author. That way the review of the code is easier when the development is done by a team.
To be sure that we know who did what, all the functions must be commented and have the author.
About testing the code there are different way to do this and sometimes unit tests are not enough and integration tests or end to end tests can be provided too.
For example on the V2 we don't do unit tests for the API but integration tests. But for the frontend we will do unit tests and end to end tests
We also try to provide a coverage that is above 50%. This could be added to the guideline.
RE: I need feedback from you about the new moderation guidelines for the development category!