Refactor condition to be easier comprehensible
The following discussion from !3366 should be addressed:
-
@cwiesner started a discussion: return unless current_customer && !params.key?(:edit_namespace) && gitlab_com_lead
nitpick (non-blocking): Not for this merge request but we should refactor this condition to make it easier comprehensible.
Proposal
This can be done by using an if
instead of an unless
which is easier to understand on a quick look.
Edited by Corinna Gogolok