-
Notifications
You must be signed in to change notification settings - Fork 554
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
Now we can do SimpleCov.add_group "Group", ["path", "other/path"] #104
Conversation
…array with a bunch o paths.
Thanks for your submission, you have not been forgotten :) I'm considering modifying the API for this so the filters and groups accept argument splats, so the line would look like this: I hope I'll have the time to dive into this soon. |
Any movement on this? Would be nice to be able to provide multiple locations for the same group |
+1 |
2 similar comments
+1 |
+1 |
Would you mind rebasing this off of master? |
689480e
to
ec43ea9
Compare
c7c5ad5
to
4c39fa3
Compare
Added feature from #104 allowing a array to be passed in a filter
fd640fa
to
739cf4d
Compare
Since #375 was merged and recognized as the new version of this PR, can this one be closed? |
@onebree Yes, I believe so. |
This commit makes it possible to define a group with multiple directories.
SimpleCov.configuration do
add_group "Units", ["app/models", "lib", "app/helpers"]
end
By the way, great gem, very easy to read, well commented.