Accept simple freezed constants in Layout/ClassStructure
and correctly handle class methods
#11329
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR fixes 2 problems:
.freeze
constants are considered as "dynamic constants" and are ignored from the consideration. Changed that.This PR also fixes #8034.
As a follow up to this, I will extend this cop to be able to detect private constants and class methods, to avoid having disables like
rubocop/lib/rubocop/cop/base.rb
Lines 330 to 333 in 20b32e4
Layout/ClassStructure
currently suggests, thats why# rubocop:disable
).