Respect DisabledByDefault in parent configs #3713
Merged
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.
d956705 changed
Config
initialization such thatConfig#deprecation_check
was called before resolving inheritance.Config#deprecation_check
causesConfig#for_all_cops
to be lazily initialized to a new hash disconnected fromConfig
's primary hash.Config#for_all_cops
would thus not reflect any settings inherited from parent configs.Essentially, it's an error for
Config#for_all_cops
to be called before inheritance is resolved. Perhaps this code could be made more robust. For now I've just rearranged config initialization such that things happen in the same order as prior to d956705./cc @NickLaMuro
Before submitting the PR make sure the following are checked:
[Fix #issue-number]
(if the related issue exists).master
(if not - rebase it).and description in grammatically correct, complete sentences.
rake generate_cops_documentation
(required only when you've added a new cop or changed the configuration/documentation of an existing cop).