Follow-up from "Refactor tracking of MR title, desc, and draft status"
Reduce ABC complexity in app/services/merge_requests/update_service.rb
:
The following discussion from !54740 (merged) should be addressed:
-
@kerrizor started a discussion: (+1 comment) We are just squeaking over the complexity limit here. We're doing a ton of work in this method, and I think there's future work for refactoring here to get it down. For now, though, let's remove the limit and follow up later.
(When we get to final approval, let's open a follow up issue from this comment.)