Skip to content

Allow rubocop exception for CurrentSettings

Rutger Wessels requested to merge 480667-approve-exception-current-organization into master

What does this MR do and why?

We have a rubocop rule that restricts the use of Current.organization to the request layer (controllers mostly). There is one valid use case: Gitlab::CurrentSettings: If we would pass the Current Organization ID in this class, we would need to modify all 1700 usages of this class.

So this MR will add the Gitlab::CurrentSettings class to the list of files and directories that can use Current::Organization

MR acceptance checklist

Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Related to #480667 (closed)

Edited by Rutger Wessels

Merge request reports