Skip to content

RegularEventControllerのリファクタリング #9543

@yokomaru

Description

@yokomaru

概要

  • 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

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions