- edited description
- changed title to No unapproval on PR where only commit message has changed
- marked as minor
No unapproval on PR where only commit message has changed
Issue #34
invalid
EDIT: my first testing failed due to human error.
Apparently with Auto Unapprove Pull Requests version 3.0.2:
A forced update changing existing commits on a Pull Request (PR) did not trigger automatic un-approval.
The diff of the PR did not change, but commit messages did change. A developer reported this to me as a bug.
Comments (4)
-
reporter -
This is expected behaviour as a result of issue #14. The plugin does not unapprove reviewers if the changes did not change the diff that has been reviewed.
-
You may want to watch this suggestion which asks for an option to unapprove on any commit, not just meaningful ones.
-
- changed status to invalid
- Log in to comment
Reproducible with Auto Unapprove Pull Requests version 3.0.2:
A forced update changing existing commits on a Pull Request (PR) behaves as follows: