-
-
Notifications
You must be signed in to change notification settings - Fork 272
Enable non-default locations of checked files #234
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
Comments
@ustimchik: consider adding this to your inherit_mode:
merge:
- Exclude
- Include
AllCops:
Include:
- lib/path/to/your/engines/**/*.rb |
@ianyamey, it seems that this config somehow conflicts with Finds 36 require:
- ./linters/custom_cops.rb
- rubocop-rails
inherit_mode:
merge:
- Exclude
- Include Still finds 36 require:
- ./linters/custom_cops.rb
- rubocop-rails
inherit_mode:
merge:
- Exclude
- Include
AllCops:
Include:
- 'plugins/**/*.rb' Only with specific rule overwrite it finds 50 require:
- ./linters/custom_cops.rb
- rubocop-rails
inherit_mode:
merge:
- Exclude
- Include
Rails/HasAndBelongsToMany:
Include:
- 'plugins/**/*.rb' |
I'm having this issue as well. It may be more straightforward to include engine paths (ex. |
I am also having this issue, not with Engines but with Packwerk, which uses a similar subfolder structure - in my case it looks like The only solution I've found so far is to copy my root-level |
When using engines, the path for controllers, models, etc. are different from the core application which forces gem users to enable "Include" for all cops with this configurable option.
It would be great if there was a possibility to have some configurable global parameters, for instance:
The text was updated successfully, but these errors were encountered: