Stop triggering mergeRequestMergeStatusUpdated subscription when MR is marked as checking
Background
A MR is marked as checking
before we check for mergeability but the UI doesn't really differs from when the MR merge_status
is unchecked
. We only allow to transition merge_status
when MR is marked as unchecked so it's not useful to update the subscribers when it happens. We can save some SQL queries if we don't trigger the subscription in this case.
Proposal
Remove checking
state in the after_transition
block in MergeRequest
that triggers the mergeRequestMergeStatusUpdated
subscription.
Edited by Patrick Bajao