Hello all,
For cockpit's master branch I just refined the branch protection rules to enable this:
Dismiss stale pull request approvals when new commits are pushed New reviewable commits pushed to a matching branch will dismiss pull request review approvals.
This is something which we already agreed to follow, and this should formalize it and automatically remove approvals on force pushes.
Please speak up if this causes any trouble.
For cockpit-{podman,ostree} I restricted the rules even more to *require* green tests for landing, and completely disallow direct pushes to master. In cockpit.git we aren't quite there yet, though.
Thanks,
Martin
cockpit-devel@lists.fedorahosted.org