Skip to content

Stop triggering subscription when MR marked as checking

Patrick Bajao requested to merge 398121-stop-triggering-checking-merge-status into master

What does this MR do and why?

Since there is no difference in the UI between unchecked and checking merge status, there's no use to trigger the subscription when MR is marked as checking. We only allow to transition a MR to checking when it is from unchecked so the UI state before the MR gets marked as checking is still valid.

This removes the checking and cannot_be_merged_rechecking state from the after_transition block that triggers the mergeRequestMergeStatusUpdated subscription.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Related to #398121 (closed)

Merge request reports