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.

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

Support nested config requires #9

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

joshpencheon
Copy link

Passing the extension's config through ConfigLoader.load_file, rather than using the private load_yaml_configuration method, ensures that require / inherit_from declarations are handled properly.

By loading the extensions additional Cop definitions prior to loading the config allow extensions to then "dogfood" by using the new cops/behaviours in their own RuboCop config.

Also allows e.g. extensions to dogfood their own enhanced config
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant