Refactor SyncAnyMergeRequestRulesService
The following discussion from !133996 (merged) should be addressed:
-
@sashi_kumar started a discussion: suggestion we seem to require only the ids of both violated and unviolated policies and instead of doing
.pluck(:id)
multiple times, would it make sense to do it once and pass the ids alone?🤔