Skip to content

Fix Rails/ApplicationController offenses

Vitali Tatarintev requested to merge 239356-fix-Rails/ApplicationController into master

What does this MR do and why?

Fixes Rails/ApplicationController offenses.

Related to #239356

It looks like the remaining ActionController::Base usages are there for a reason. We don't want to use a load of extra functionality in some controllers. Thus, disable the cop in place for those controllers. That idea was discussed in the thread below.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Vitali Tatarintev

Merge request reports