MRs don't show all merge errors

MergeRequests::MergeService#log_merge_error doesn't actually update the MR's merge_error field, which means any parts that use the common return log_merge_error(message) unless merge_request.predicate? idiom will just show a permanently spinning 'merge in progress' status, rather than the actual error.