runbot/runbot_merge/changelog/2025-02/merge-method-acl.md

IMP: allow non-reviewer employees to set the merge method on the PRs they create

Previously this was only allowed for reviewers actual (note: not necessarily
the PR's own), but when the reviewer is busy and the PR is blocked on just the
merge method it can be frustrating. However because the merge method has commit
impliciation via the PR description (squash/merge/rebase-merge) giving access to
any author might be a bit too relaxed.
[IMP] runbot_merge: relax requirements for setting the merge method Not entirely sure about just allowing any PR author to set the merge method as it gives them a lot more control over commits (via the PR message), and I'm uncertain about the ramifications of doing that. However if the author of the PR is classified as an employee (via a provisioned user linked to the github account) then allow it. Should not be prone to abuse at least. Fixes #1036 2025-01-30 17:47:18 +07:00			`IMP: allow non-reviewer employees to set the merge method on the PRs they create`

			`Previously this was only allowed for reviewers actual (note: not necessarily`
			`the PR's own), but when the reviewer is busy and the PR is blocked on just the`
			`merge method it can be frustrating. However because the merge method has commit`
			`impliciation via the PR description (squash/merge/rebase-merge) giving access to`
			`any author might be a bit too relaxed.`