Follow-up from "`current_application_settings` belongs on `Gitlab::CurrentSettings`"
The following discussion from !13955 (merged) should be addressed:
-
@rymai started a discussion: From a semantic perspective, I think
Gitlab::CurrentSettings.help_page_support_url.presence
would make more sense (no need to repeat the "current settings" part since it's already the module name)?
What do you think @smcgivern?