dismiss stale pull request approvals when new commits are pushed
include administrators
This changes reflect our current practices in EngX and InfraEng repos. While code review is required, there are edge cases in which a PR must be merged prior to review. Similarly, dismissing stale approvals should be at the PR author's discretion, as some changes (e.g., rebases) do not require a follow-up review.
This removes two recommended branch settings:
This changes reflect our current practices in EngX and InfraEng repos. While code review is required, there are edge cases in which a PR must be merged prior to review. Similarly, dismissing stale approvals should be at the PR author's discretion, as some changes (e.g., rebases) do not require a follow-up review.