Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Metrics/AbcSizeを15から17に緩めたい #28

Closed
lastgleam opened this issue Sep 9, 2020 · 2 comments · Fixed by #30
Closed

Metrics/AbcSizeを15から17に緩めたい #28

lastgleam opened this issue Sep 9, 2020 · 2 comments · Fixed by #30

Comments

@lastgleam
Copy link
Contributor

lastgleam commented Sep 9, 2020 •
edited

対象規約

Metrics/AbcSize

種類

修正

理由

制限が厳しい

個人的にたった条件式1~2行追加で怒られて（特にFeatureToggle仕込む時）
リファクタリングしないといけないことになってしまうケースが何回かあった。

# rubocop: disable Metrics/AbcSizeが多い

IDEを使っていなくてrubocopを手動で回している場合、実装終えてからAbcSizeを指摘されると
書いたコードを全体的に見直さないといけなくなり修正に負担がかかってしまう。
その修正を諦めてMetrics/AbcSizeをdisableしてしまうのも多々ある。
なるべくdisableするのを減らしたいし。

備考

ちょうどrubocop本体もMetrics/AbcSizeを17にするPRがmasterブランチにマージされた。
rubocop/rubocop#8037
@ujihisa
Copy link
Contributor

ujihisa commented Sep 9, 2020

image

@ujihisa
Copy link
Contributor

ujihisa commented Sep 9, 2020

s/FeatureToggle/FeatureToggles/

@lastgleam lastgleam changed the title [WIP] Metrics/AbcSizeを15から17に緩めたい Metrics/AbcSizeを15から17に緩めたい Sep 9, 2020
@lastgleam lastgleam mentioned this issue Sep 25, 2020
Closed
@ysk1180 ysk1180 mentioned this issue Oct 6, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@ujihisa @lastgleam