Follow-up from "Conditional partial rendering for EE in application settings"

The following discussion from !7214 (merged) should be addressed:

  • @godfat started a discussion: (+2 comments)

    This could be just me, but it's a bit hard for me to understand unless with ||. What do you think about using if instead?

    - return if !Gitlab.com? && !Rails.env.development?

    I'll leave this up to you and the maintainer to decide.

Assignee Loading
Time tracking Loading