If I dismiss a “Request Changes” review, I get to enter a comment.
It’s later impossible to edit this comment after submission.
This makes it impossible to fix possible logic errors, grammar issues, …
(I assume that similar issues exist with “Approve” reviews)
The review system (review comment / dismissal comment) should be integrated better, to allow the same features as for comments (edit / deletion) under almost any circumstance. Currently, there’s a lot of confusing “rules”, like the inability to delete a review comment which “Requested Changes”.
In the greater scheme of things:
Ideally the review system was redesigned to be part of the normal comment system where comments could be tagged as “[X] Requesting changes” / “[X] Approving changes”. A review could be dismissed by removing that tag, or deleting the comment. Any review comment could be edited at all times.
The number of comments with review tags would have to be shown in the PR list (similar how review status is shown in the PR list).
I believe that this would replace the current review system with a more powerful system, that’s more intuitive to use.