-
Suggestion
-
Resolution: Unresolved
-
None
-
None
-
3
-
When dev1 has opened a pull request from e.g. a feature branch to the develop branch, then dev2 has approved the pull request, and afterwards dev1 pushes another change to the branch, the pull request is marked as unapproved. This is sensible behaviour, but there is one thing about it that I don't like, and which leads to confusion: It is not obvious, that the "Unapproved" was an automatic action done by the system. It looks like the person, who had previously approved the pull request, had now manually unapproved it again, without providing a comment.
IMO it would be much better if there would be an additional remark (e.g. instead of UNAPPROVED, the label could read UNAPPROVED AUTOMATICALLY or something).