概要
RegularEventsControllerが肥大化しており、Metrics/ClassLengthに引っかかる状態になっている
- 1/14のMTGにてkomagataさんに確認し、現在進行中のPRでは暫定対応として
rubocop:disableMetrics/ClassLengthを追加して先に進める
- controller内のロジックを整理し、必要に応じてmodel等へ移動してRubocop違反を解消する
期待される振る舞い
RegularEventsControllerからMetrics/ClassLength違反を解消し、rubocop:disable Metrics/ClassLength を削除する
- 可能であれば今後の変更で違反が再発しにくい構造にする
関連Issue
- 現在進行中の以下2つのIssueはいずれも
RegularEventsControllerにロジック追加を含むが、これらのPR内ではリファクタリングは行わず別途このIssueを起点に行う
Metrics/ClassLengthの一時対応として両PRで暫定的に以下を追加する
class RegularEventsController < ApplicationController # rubocop:disable Metrics/ClassLength
- 先にマージされたPRをmainに取り込む
- もう片方のPRはmainをrebaseして取り込み競合や差分を解消する
対象Issue
概要
RegularEventsControllerが肥大化しており、Metrics/ClassLengthに引っかかる状態になっているrubocop:disableMetrics/ClassLengthを追加して先に進める期待される振る舞い
RegularEventsControllerからMetrics/ClassLength違反を解消し、rubocop:disable Metrics/ClassLengthを削除する関連Issue
RegularEventsControllerにロジック追加を含むが、これらのPR内ではリファクタリングは行わず別途このIssueを起点に行うMetrics/ClassLengthの一時対応として両PRで暫定的に以下を追加する対象Issue