handle directories as cmdline args
-
Kevin Rutherford
In fact this is what Reek 0.2 used to do and I took it out. Let's do some design. If a commandline argument is a directory Reek should look at:
- all files called DIR/*.rb ?
- all files called DIR/*/.rb ?
So should this be a config-file option? What about a list of files to be excluded (eg. *_spec.rb)?
Currently Reek::RakeTask has an option to specify the set of files, and globs are permitted. So in a way this is already supported. Plus, most operating systems provide a way to find loads of files and pass them to the commandline, eg:
$ reek `find DIR -name \*.rb -print` -
Kevin Rutherford
- State changed from new to resolved
Closed, because there are already a couple of viable alternatives.
-
Kevin Rutherford
"app/models".should_not reekis very sexy. Is
"app/models/**/*.rb".should_not reekany less so?
-
Kevin Rutherford
Yes, I'll add some more factory methods to Source. In a few days...
Please Sign in or create a free account to add a new ticket.
With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.