-
Notifications
You must be signed in to change notification settings - Fork 225
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
Settings file #30
Comments
I agree, I think a |
In debt_ceiling I use the Confirguations gem, and read in a ruby file with rails like configuration block... I think I prefer that to yml, but agree having settings/config is a good idea. |
+1. It would be great to specify in a config file which folders ruby critic should critique. |
Agree, would be useful. Would somebody like to take a stab at this? ;) |
The same comment I wrote for #11 applies here:
;) |
I am working on this, using yml ( |
@nav16 Any updates on this? I have a project right now where we patch a gem and it is very picky about how we patch it. This file is the only thing that's not an 'A' and is keeping me from doing a |
It would be nice if the rubycritic config was basically just an extended version of the Reek config — so we could copy our current Reek configuration into it, and add other configuration options for the other tools (flog, flay, etc.) and for rubycritic itself. Or maybe provide a rubycritic config directory that can contain config files specific to each tool (e.g., In particular, I want to be able to configure skipping some of the checks from Flog and Flay (e.g., Right now, my rubycritic my report gets filled up with lots of complexity and duplicate code smells from my spec directory, unless I don’t include the spec directory (and miss out on reporting the code smells I do want to see for specs). Thanks! |
I guess this can be closed, since we can use |
Instead of having settings purely as flags, it would be interesting to have a configuration file, which could be ported and used across several projects.
The text was updated successfully, but these errors were encountered: