Rename can_be_approved_by? to be more descriptive
What does this MR do and why?
can_be_approved_by?
can be confusing since we have a policy check called :approve_merge_request
and it can be called like user.can?(:approve_merge_request, merge_request, user)
This is an attempt to make it more clear how this method is different from that policy check.
Related to #366529 (closed) and !95021 (comment 1058046730)
MR acceptance checklist
This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.
-
I have evaluated the MR acceptance checklist for this MR.
Related to #366529 (closed)
Edited by Sincheol (David) Kim